From patchwork Fri Mar 9 13:07:18 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bryan O'Donoghue X-Patchwork-Id: 883608 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=lists.denx.de (client-ip=81.169.180.215; helo=lists.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="LC6ID3Za"; dkim-atps=neutral Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 3zySN803qYz9sbh for ; Sat, 10 Mar 2018 00:07:35 +1100 (AEDT) Received: by lists.denx.de (Postfix, from userid 105) id 84527C21D83; Fri, 9 Mar 2018 13:07:28 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de X-Spam-Level: X-Spam-Status: No, score=-0.0 required=5.0 tests=RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.denx.de (localhost [IPv6:::1]) by lists.denx.de (Postfix) with ESMTP id BF882C21C51; Fri, 9 Mar 2018 13:07:25 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id 0F526C21BE5; Fri, 9 Mar 2018 13:07:25 +0000 (UTC) Received: from mail-wm0-f66.google.com (mail-wm0-f66.google.com [74.125.82.66]) by lists.denx.de (Postfix) with ESMTPS id AE954C21C38 for ; Fri, 9 Mar 2018 13:07:24 +0000 (UTC) Received: by mail-wm0-f66.google.com with SMTP id t6so3783264wmt.5 for ; Fri, 09 Mar 2018 05:07:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=oIhTZQ8SNqhh00xSe7RjtvimWVxQLVUmJdOf2E5xdVc=; b=LC6ID3ZaxyBVa6LhdmrXVPVMcTCUf5cQiYVs9RsTET81yIwCaHfegqQfa6U6uqv60Z Kvz+q8xZMDHIXVgUS14bQwl2dGih0hTOtLgjRLAa0iOvw2xcOtaLwrR1gRR/BXqp5Ak2 9ehpuNQsQ0xJdMu7wVD1fQ5GaeQD56ystfzOo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=oIhTZQ8SNqhh00xSe7RjtvimWVxQLVUmJdOf2E5xdVc=; b=oiRcxjlXYwW3HJbt8RbvgmvnSB9JPQ/Ko/Zn96RymlJ/PG75urXMPt6euzg8ii50r8 bHg3HfTzsUgCkFIzYaMIUlCsR6Az6y0ogpwc/mkRmTcQXik2fQO1obH8J5PNEJlr2G/f mc5zZA+6kCgPNtpuyUahEpt7+SIAXhl6pySz3Y1lD3CeF1RUaHljJ0gJf/PzaXNSgCBi 3Ko8lF6aPQNyI7zWkf4d0W0iqwB2oYJMFAy4TckKZ0/VgMvMqR3K2xLxAUApgJHSXR6r 6PSwt6reurmw1bs1hA++7zIDB36RiRDOorEZWJ7tmEHDwwnHHUzy84OdXZ0FWfnnhkkB ghKg== X-Gm-Message-State: AElRT7HKB7qG07WgPxhR9P5e1HWFCdzuoqxQHKN+hYSFSmYNXGU5+V15 8I7jiwCHD96psOkCQat9npGqMOUUD8A= X-Google-Smtp-Source: AG47ELtmzUIn8H0bCjFtrVt06t4rThXJrVseDyQhxNCWkpMIOpsB85JNNsbXxYO+RR0EeLxe761sTA== X-Received: by 10.80.186.25 with SMTP id g25mr13021896edc.28.1520600843922; Fri, 09 Mar 2018 05:07:23 -0800 (PST) Received: from localhost.localdomain ([109.255.42.2]) by smtp.gmail.com with ESMTPSA id b11sm827527edc.10.2018.03.09.05.07.22 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 09 Mar 2018 05:07:23 -0800 (PST) From: Bryan O'Donoghue To: U-Boot@lists.denx.de Date: Fri, 9 Mar 2018 13:07:18 +0000 Message-Id: <1520600841-8810-1-git-send-email-bryan.odonoghue@linaro.org> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 Subject: [U-Boot] [PATCH 0/3] HAB Fixes for 2018.03-rc4 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.18 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" This set fixes a few errors and gotchas I've found when rebasing my project on top of 2018-rc4. First is a breakage with the Linaro compiler -> gcc-linaro-7.2.1 which is a simple one. Second is consistency of the __packed attribute - more of a housekeeping activity. Last patch fixes a behaviour that has been introduced without I believe fully thinking through the impact on downstream users. If we force the DCD to be NULL for olders SoCs - that means people have to go out and re-sign all of their binaries if they are updating u-boot. It should be possible to update u-boot in isolation without forcing update of signed binaries u-boot authenticates via HAB. If newer NXP SoCs rely on the DCD being NULL then either 1. Just let the authenticate_image() call fail It can return an error - this should "just work" if the SoC will reject an API callback with the DCD non-NULL then - let it. 2. If the SoC actually has a bug that _requires_ the DCD to be NULL. Then add a check into u-boot to check SoC versions. In either case older SoCs should "just work" and it should be possible to update u-boot without having to update all associated signed binaries. Please apply before next official u-boot release. Bryan O'Donoghue (3): imx: hab: Fix usage of packed attribute imx: hab: Make usage of packed attribute consistent imx: hab: Convert DCD non-NULL error to warning arch/arm/include/asm/mach-imx/hab.h | 5 +++-- arch/arm/mach-imx/hab.c | 6 ++---- 2 files changed, 5 insertions(+), 6 deletions(-)