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

[Michael Nazzareno Trimarchi]

Hi

I will add to my queue that I have now with patches, I have already
prepared a branch

Michael

On Thu, Oct 9, 2025 at 5:11 PM Tom Rini <trini at konsulko.com> wrote:
>
> 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



-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com