On 19. 08. 20 15:15, Tom Rini wrote: > 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.
This one is easy but others can be problematic. But there is serious issue with these symbols. When you run make randconfig you see unfortunately a lot of issues reported. Do you use any tool/script for defconfig update when Kconfig layout has changed? Thanks, Michal
signature.asc
Description: OpenPGP digital signature