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

[Eugen Hristev]

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 ?