public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Sughosh Ganu <sughosh.ganu@arm.com>
To: Kory Maincent <kory.maincent@bootlin.com>
Cc: u-boot@lists.denx.de,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	thomas.petazzoni@bootlin.com, Tom Rini <trini@konsulko.com>
Subject: Re: [PATCH] fwu-mdata: Allow multiple metadata storage drivers to be enabled
Date: Thu, 12 Feb 2026 16:47:47 +0530	[thread overview]
Message-ID: <aY222wFN_8GWkAJf@a079122.arm.com> (raw)
In-Reply-To: <20260114095654.736511-1-kory.maincent@bootlin.com>

On Wed, Jan 14, 2026 at 10:56:53AM +0100, Kory Maincent wrote:
> Change the Kconfig from a "choice" to a conditional block, allowing
> multiple FWU metadata storage drivers to be selected simultaneously
> instead of being mutually exclusive.
> 
> This enables systems with FWU metadata on different storage types
> (e.g., both GPT-partitioned block devices and MTD devices) to have
> both drivers compiled in. The board can then select the appropriate
> driver at runtime based on the devicetree description.
> 
> The change converts FWU_MDATA to a menuconfig and replaces the
> "choice/endchoice" block with "if FWU_MDATA/endif", making
> FWU_MDATA_GPT_BLK default to 'y' for backward compatibility.
> 
> Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
> ---

Acked-by: Sughosh Ganu <sughosh.ganu@arm.com>

-sughosh

>  drivers/fwu-mdata/Kconfig | 12 +++++-------
>  1 file changed, 5 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/fwu-mdata/Kconfig b/drivers/fwu-mdata/Kconfig
> index 42736a5e43b..59571ac01ea 100644
> --- a/drivers/fwu-mdata/Kconfig
> +++ b/drivers/fwu-mdata/Kconfig
> @@ -1,4 +1,4 @@
> -config FWU_MDATA
> +menuconfig FWU_MDATA
>  	bool "Driver support for accessing FWU Metadata"
>  	depends on DM
>  	help
> @@ -6,16 +6,14 @@ config FWU_MDATA
>  	  FWU Metadata partitions reside on the same storage device
>  	  which contains the other FWU updatable firmware images.
>  
> -choice
> -	prompt "Storage Layout Scheme"
> -	depends on FWU_MDATA
> -	default FWU_MDATA_GPT_BLK
> +if FWU_MDATA
>  
>  config FWU_MDATA_GPT_BLK
>  	bool "FWU Metadata access for GPT partitioned Block devices"
>  	select PARTITION_TYPE_GUID
>  	select PARTITION_UUIDS
> -	depends on FWU_MDATA && BLK && EFI_PARTITION
> +	depends on BLK && EFI_PARTITION
> +	default y
>  	help
>  	  Enable support for accessing FWU Metadata on GPT partitioned
>  	  block devices.
> @@ -28,4 +26,4 @@ config FWU_MDATA_MTD
>  	  (or non-GPT partitioned, e.g. partition nodes in devicetree)
>  	  MTD devices.
>  
> -endchoice
> +endif
> -- 
> 2.43.0
> 

      parent reply	other threads:[~2026-02-12 11:17 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-14  9:56 [PATCH] fwu-mdata: Allow multiple metadata storage drivers to be enabled Kory Maincent
2026-01-14 14:49 ` Tom Rini
2026-02-06 14:19   ` Kory Maincent
2026-02-11  9:04     ` Ilias Apalodimas
2026-02-11 10:12       ` Sughosh Ganu
2026-02-11 10:22         ` Ilias Apalodimas
2026-02-12 11:17 ` Sughosh Ganu [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aY222wFN_8GWkAJf@a079122.arm.com \
    --to=sughosh.ganu@arm.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=kory.maincent@bootlin.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox