From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2E98BE9A047 for ; Wed, 18 Feb 2026 11:23:35 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 5C64983CB9; Wed, 18 Feb 2026 12:23:33 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: by phobos.denx.de (Postfix, from userid 109) id E7D0283B99; Wed, 18 Feb 2026 12:23:31 +0100 (CET) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by phobos.denx.de (Postfix) with ESMTP id 6FCE383936 for ; Wed, 18 Feb 2026 12:23:28 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=sughosh.ganu@arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DECDE1477; Wed, 18 Feb 2026 03:23:20 -0800 (PST) Received: from localhost (a079122.arm.com [10.164.21.38]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 6B2BA3F7F5; Wed, 18 Feb 2026 03:23:26 -0800 (PST) Date: Wed, 18 Feb 2026 16:53:23 +0530 From: Sughosh Ganu To: Kory Maincent Cc: u-boot@lists.denx.de, jose.marinho@arm.com, Thomas Petazzoni , Tom Rini , Patrice Chotard , Paul HENRYS , Greg Malysa , Arturs Artamonovs , Vasileios Bimpikas , Utsav Agarwal , Nathan Barrett-Morrison , Peng Fan , Simon Glass , Duje =?utf-8?Q?Mihanovi=C4=87?= , Stefan Roese , Mattijs Korpershoek , Sumit Garg , Heiko Schocher , Alif Zakuan Yuslaimi , E Shattow , Raymond Mao , Jan Kiszka , Shiji Yang , Daniel Golle , Heinrich Schuchardt , Ilias Apalodimas , Leonard Anderweit , Yao Zi Subject: Re: [PATCH v3 5/6] tools: Add support for fwumdata tool Message-ID: References: <20260216-feature_fwumdata-v3-0-9ecc5b10456d@bootlin.com> <20260216-feature_fwumdata-v3-5-9ecc5b10456d@bootlin.com> <20260218113116.3c447668@kmaincent-XPS-13-7390> <20260218120816.760df1ff@kmaincent-XPS-13-7390> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260218120816.760df1ff@kmaincent-XPS-13-7390> X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean On Wed, Feb 18, 2026 at 12:08:16PM +0100, Kory Maincent wrote: > On Wed, 18 Feb 2026 16:15:14 +0530 > Sughosh Ganu wrote: > > > On Wed, Feb 18, 2026 at 11:31:16AM +0100, Kory Maincent wrote: > > > Hello Sughosh, > > > > > > On Wed, 18 Feb 2026 15:26:35 +0530 > > > Sughosh Ganu wrote: > > > > > > > On Wed, Feb 18, 2026 at 03:17:58PM +0530, Sughosh Ganu wrote: > > > > > On Mon, Feb 16, 2026 at 02:35:35PM +0100, Kory Maincent wrote: > > > > > > Add a new fwumdata tool to allows users to read, display, and modify > > > > > > FWU (Firmware Update) metadata from Linux userspace. It provides > > > > > > functionality similar to fw_printenv/fw_setenv but for FWU metadata. > > > > > > Users can view metadata, change active/previous bank indices, modify > > > > > > bank states, and set image acceptance flags. Configuration is done > > > > > > via fwumdata.config file. > > > > > > > > > > > > Signed-off-by: Kory Maincent > > > > > > --- > > > > > > > > > > I am not sure if this has been discussed with Ilias earlier, and > > > > > apologies if it has been, but I do think that this patch is not > > > > > adhering to the DEN0118 specification, specifically the part mentioned > > > > > in section A3.2.1, which says that the metadata is to be maintained by > > > > > the Update Agent. I would like to hear from Jose Marinho, who is the > > > > > author of the spec, on what he thinks about this approach. > > > > > > > > > > I do think that the other patches in series look fine, and can be > > > > > applied. There is just an issue of inclusion of the tool for > > > > > building. I will comment on that patch separately. > > > > > > > > > > > > Adding Jose to the discussion. > > > > > > We indeed already had some discussion about it with Ilias. > > > https://lists.denx.de/pipermail/u-boot/2025-December/605924.html > > > > > > It seems ST deviate from the original idea behind the standard. It does not > > > use EFI neither update capsule. > > > TF-A is managing the selection of the boot partition (FIP image: OPTEE + > > > bootloader) according to the FWU metadata content and is dealingwith the > > > rollback mechanism. Linux through this tools is the update agent to select > > > the boot image. > > > > Okay. I guess that is fine then -- with the Update Agent residing in > > Linux, this is not an issue. My next question then would be, why are > > you adding this to U-Boot then :). Can this not be bundled with a > > related package of RAUC maybe(if one such exists)? > > There was also a discussion about this topic: > https://lists.denx.de/pipermail/u-boot/2025-December/604901.html > > The final conclusion is: > - for the short/medium term we add it alongside the mkfwumdata tool. > - for the long term it would be nice to extract all generic tools to a different repo > than U-boot to make life easier for other projects. Okay. In that case, please post a v4 which fixes the building of the tools. Thanks. -sughosh