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.si...@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.s...@xilinx.com> > >> AuthorDate: Thu Dec 19 02:27:42 2019 -0700 > >> Commit: Michal Simek <michal.si...@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.s...@xilinx.com> > >> Signed-off-by: Michal Simek <michal.si...@xilinx.com> > > > > Ah, I misunderstood the comment, sorry. Yes, this is right then, > > thanks. > > > > Reviewed-by: Tom Rini <tr...@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
signature.asc
Description: PGP signature
