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> -- Tom
signature.asc
Description: PGP signature