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 <[email protected]>
> >>> ---
> >>> Cc: Eugen Hristev <[email protected]>
> >>> Cc: Dario Binacchi <[email protected]>
> >>> Cc: Michael Trimarchi <[email protected]>
> >>> ---
> >>>  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/[email protected]/
> > 
> 
> Then maybe Dario can pick this up together with the dependency patch ?

Works for me, Dario?

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to