[PATCH 4/9] nand: Kconfig: Change dependency for NAND_ARASAN

[Tom Rini]

On Wed, Aug 19, 2020 at 02:52:11PM +0200, Michal Simek wrote:
> 
> 
> On 19. 08. 20 14:43, Tom Rini wrote:
> > On Wed, Aug 19, 2020 at 02:36:20PM +0200, Michal Simek wrote:
> >> On 19. 08. 20 14:31, Tom Rini wrote:
> >>> On Wed, Aug 19, 2020 at 10:44:18AM +0200, Michal Simek wrote:
> >>>
> >>>> NAND_ARASAN selecting DM_MTD uunconditionally. Driver can be enabled with
> >>>> !DM that's why Kconfig it showing it as error:
> >>>>
> >>>> WARNING: unmet direct dependencies detected for DM_MTD
> >>>>   Depends on [n]: DM [=n]
> >>>>   Selected by [y]:
> >>>>   - NAND_ARASAN [=y] && MTD_RAW_NAND [=y]
> >>>>
> >>>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> >>>> ---
> >>>>
> >>>>  drivers/mtd/nand/raw/Kconfig | 2 +-
> >>>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig
> >>>> index 06b2ff972cad..df4cbd52cf5c 100644
> >>>> --- a/drivers/mtd/nand/raw/Kconfig
> >>>> +++ b/drivers/mtd/nand/raw/Kconfig
> >>>> @@ -233,7 +233,7 @@ endif
> >>>>  config NAND_ARASAN
> >>>>  	bool "Configure Arasan Nand"
> >>>>  	select SYS_NAND_SELF_INIT
> >>>> -	select DM_MTD
> >>>> +	depends on DM_MTD
> >>>>  	imply CMD_NAND
> >>>>  	help
> >>>>  	  This enables Nand driver support for Arasan nand flash
> >>>
> >>> If we can use the driver without DM then we can't depend nor select
> >>> DM_MTD.  Is someone using the driver still without DM_MTD?
> >>>
> >>
> >> Driver has been converted to DM already. It is used by ZynqMP platforms.
> >> And it can't be used without DM_MTD.
> >>
> >> commit b014b833bd627d27542b50e71df9cf0628f241e7
> >> Author:     Ashok Reddy Soma <ashok.reddy.soma at xilinx.com>
> >> AuthorDate: Thu Dec 19 02:27:42 2019 -0700
> >> Commit:     Michal Simek <michal.simek at xilinx.com>
> >> CommitDate: Tue Jan 14 09:05:52 2020 +0100
> >>
> >>     mtd: nand: Move arasan nand driver to driver model
> >>
> >>     Make changes to arasan nand driver to move it to driver model.
> >>     Select DM_MTD if arasan nand driver is selected.
> >>
> >>     Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma at xilinx.com>
> >>     Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> > 
> > Ah, I misunderstood the comment, sorry.  Yes, this is right then,
> > thanks.
> > 
> > Reviewed-by: Tom Rini <trini at konsulko.com>
> > 
> 
> ok. I just need to enable this symbol for zynqmp platforms.

That is the hard part about all of these patches, when you go from
"select X" to "depends on X", you also need to run a script and make
sure that every config that enables the main symbol (NAND_ARASAN) also
enables the now depended on symbol (DM_MTD) and sync the configs after
the change.

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