[U-Boot] [PATCH 2/6] mmc: complete the unfinished move of CONFIG_MMC

Masahiro Yamada yamada.masahiro at socionext.com
Wed Dec 7 02:45:34 CET 2016


Hi Tom,


2016-12-06 22:03 GMT+09:00 Tom Rini <trini at konsulko.com>:

>
>> I tried your suggestion.
>> It is true that 625 touched defconfigs were decreased to 389,
>> but do you think this is a really good idea?
>>
>> Theoretically, nothing of MMC
>> is related to ARM, PowerPC, Sandbox.
>>
>> I think this is shortening defconfigs
>> for the purpose of shortening.
>
> Yes, it is for the purpose of shortening the configs.
>
>> BTW, do you know that
>> Kbuild subsystem is discussing a new keyword "imply" ?
>>
>> https://patchwork.kernel.org/patch/9422463/
>>
>>
>> I am thinking it, if accepted, will be a better solution than
>> such unrelated "depends on <ARCH>" or "depends on <SOC>".
>
> Adding "imply" could help here I think, yes.  The problem is that I want
> to make the defaults in Kconfig produce a reasonable defconfig rather
> than require each new defconfig to require a lot of information to
> produce a reasonable binary.  Looking at the imply examples I think that
> yes, we could probably get away with doing things like:
> arch/Kconfig:
> config ARM
>         bool "ARM Architecture"
>         ...
>         imply MMC
>         imply DISPLAY_BOARDINFO
>         imply DISPLAY_CPUINFO
>

Yes, this looks better.

For now, I will go with "default ARM || PPC || SANDBOX".
I will send v2.


Once the "imply" is merged in the kernel,
we can immediately sync with it.
Then, it would be easy to convert "default" to "imply".



Oh, I found a more crazy default.
The "imply" may be helpful for cleanups.

config SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
        bool "MMC raw mode: by sector"
        depends on SPL
        default y if ARCH_SUNXI || ARCH_DAVINCI || ARCH_UNIPHIER ||ARCH_MX6 || \
                     ARCH_ROCKCHIP || ARCH_MVEBU ||  ARCH_SOCFPGA || \
                     ARCH_AT91 || ARCH_ZYNQ || ARCH_KEYSTONE || OMAP34XX || \
                     OMAP44XX || OMAP54XX || AM33XX || AM43XX
        help
          Use sector number for specifying U-Boot location on MMC/SD in
          raw mode.



-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list