[PATCH] nand: atmel: Rework ATMEL_EBI and DM_NAND_ATMEL interaction

[Tom Rini]

On Thu, Oct 09, 2025 at 05:58:48PM +0300, Eugen Hristev wrote:
> 
> 
> On 10/9/25 17:26, Tom Rini wrote:
> > On Thu, Oct 09, 2025 at 12:48:29PM +0300, Eugen Hristev wrote:
> >>
> >>
> >> On 9/26/25 18:31, Tom Rini wrote:
> >>> The way that the NAND driver under DM_NAND_ATMEL is probed is by the
> >>> dummy memory driver controlled by ATMEL_EBI. Rather than require that
> >>> for NAND to work both be enabled, make NAND select ATMEL_EBI and do not
> >>> prompt for ATMEL_EBI as it only triggers the probe for NAND.
> >>>
> >>> Signed-off-by: Tom Rini <trini at konsulko.com>
> >>> ---
> >>> Cc: Eugen Hristev <eugen.hristev at linaro.org>
> >>> Cc: Dario Binacchi <dario.binacchi at amarulasolutions.com>
> >>> Cc: Michael Trimarchi <michael at amarulasolutions.com>
> >>> ---
> >>>  drivers/memory/Kconfig       | 2 +-
> >>>  drivers/mtd/nand/raw/Kconfig | 1 +
> >>>  2 files changed, 2 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
> >>> index eaee739c6aab..591d9d9c6569 100644
> >>> --- a/drivers/memory/Kconfig
> >>> +++ b/drivers/memory/Kconfig
> >>> @@ -14,7 +14,7 @@ config MEMORY
> >>>  	  For now this uclass has no methods yet.
> >>>  
> >>>  config ATMEL_EBI
> >>> -	bool "Support for Atmel EBI"
> >>> +	bool
> >>>  	help
> >>>  	  Driver for Atmel EBI controller. This is a dummy
> >>>  	  driver. Doesn't provide an access to EBI controller. Select
> >>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig
> >>> index aa4bb2a0595c..f083113486ed 100644
> >>> --- a/drivers/mtd/nand/raw/Kconfig
> >>> +++ b/drivers/mtd/nand/raw/Kconfig
> >>> @@ -50,6 +50,7 @@ config SYS_NAND_NO_SUBPAGE_WRITE
> >>>  config DM_NAND_ATMEL
> >>>  	bool "Support Atmel NAND controller with DM support"
> >>>  	depends on ARCH_AT91
> >>
> >> Which branch is this patch based on ?
> >> I have a conflict because this line ^^ is missing in master ("depends on
> >> ARCH_AT91")
> >> Is there another patch for that ?
> > 
> > Ugh, yes, sorry, this patch depends on:
> > https://patchwork.ozlabs.org/project/uboot/patch/20250711151604.2224243-1-trini@konsulko.com/
> > 
> 
> Then maybe Dario can pick this up together with the dependency patch ?

Works for me, Dario?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20251009/94c4db72/attachment.sig>