[U-Boot] [PATCH 3/4] Move CONFIG_NAND_MXS to defconfig

Jagan Teki
2017-01-19T16:34:25Z

On Thu, Jan 19, 2017 at 5:11 PM, Tom Rini <trini@konsulko.com> wrote:
> On Thu, Jan 19, 2017 at 04:46:18PM +0100, Jagan Teki wrote:
>> On Wed, Jan 11, 2017 at 12:14 AM, Tom Rini <trini@konsulko.com> wrote:
>> > On Sat, Jan 07, 2017 at 12:42:34PM +0100, Jagan Teki wrote:
>> >
>> >> Cc: Stefano Babic <sbabic@denx.de>
>> >> Signed-off-by: Jagan Teki <jagan@openedev.com>
>> > [snip]
>> >> 33 files changed, 21 insertions(+), 12 deletions(-)
>> > [snip]
>> >> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
>> >> index 65bb040..682e4e3 100644
>> >> --- a/drivers/mtd/nand/Kconfig
>> >> +++ b/drivers/mtd/nand/Kconfig
>> >> @@ -82,7 +82,6 @@ config NAND_ARASAN
>> >>
>> >> config NAND_MXS
>> >> bool "MXS NAND support"
>> >> - depends on MX6
>> >> help
>> >> This enables NAND driver for the NAND flash controller on the
>> >> MXS processors.
>> >
>> > I think we've got something going wrong here. Why are we removing the
>> > depends on line? I know we can't make this a default y until 'NAND'
>> > gets migrated, and that's non-trivial. Thanks!
>>
>> Yes, I was trying to add 'depends on' but ended-up making more
>> complex, because there is common config ARCH_MXC like Linux and
>
> I assume that should read "there is not". And then we should introduce
> ARCH_MXC like Linux and use it. Thanks!

Yeah, sorry for the typo. I too plan for adding ARCH_MXC and it will
be more change-set better to move defconfig changes first and will
update them after?

thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.