]> git.kernelconcepts.de Git - karo-tx-uboot.git/commit
dm: select CONFIG_DM* options
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Tue, 31 Mar 2015 03:47:53 +0000 (12:47 +0900)
committerLothar Waßmann <LW@KARO-electronics.de>
Tue, 8 Sep 2015 19:56:14 +0000 (21:56 +0200)
commit8e2f246ec49a75974fb6d2bb247cec8f6d02a74c
treee75734915cc6481aa41f016fc3e0541ed653f3c9
parent8ee9331ae4dd59c0027f91706ed576da313e11c2
dm: select CONFIG_DM* options

As mentioned in the previous commit, adding default values in each
Kconfig causes problems because it does not co-exist with the
"depends on" syntax.  (Please note this is not a bug of Kconfig.)
We should not do so unless we have a special reason.  Actually,
for CONFIG_DM*, we have no good reason to do so.

Generally, CONFIG_DM is not a user-configurable option.  Once we
convert a driver into Driver Model, the board only works with Driver
Model, i.e. CONFIG_DM must be always enabled for that board.
So, using "select DM" is more suitable rather than allowing users to
modify it.  Another good thing is, Kconfig warns unmet dependencies
for "select" syntax, so we easily notice bugs.

Actually, CONFIG_DM and other related options have been added
without consistency: some into arch/*/Kconfig, some into
board/*/Kconfig, and some into configs/*_defconfig.

This commit prefers "select" and cleans up the following issues.

[1] Never use "CONFIG_DM=n" in defconfig files

It is really rare to add "CONFIG_FOO=n" to disable CONFIG options.
It is more common to use "# CONFIG_FOO is not set".  But here, we
do not even have to do it.
Less than half of OMAP3 boards have been converted to Driver Model.
Adding the default values to arch/arm/cpu/armv7/omap3/Kconfig is
weird.  Instead, add "select DM" only to appropriate boards, which
eventually eliminates "CONFIG_DM=n", etc.

[2] Delete redundant CONFIGs

Sandbox sets CONFIG_DM in arch/sandbox/Kconfig and defines it again
in configs/sandbox_defconfig.
Likewise, OMAP3 sets CONFIG_DM arch/arm/cpu/armv7/omap3/Kconfig and
defines it also in omap3_beagle_defconfig and devkit8000_defconfig.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
43 files changed:
arch/Kconfig
arch/arm/Kconfig
arch/arm/cpu/armv7/exynos/Kconfig
arch/arm/cpu/armv7/omap3/Kconfig
arch/arm/mach-bcm283x/Kconfig
arch/arm/mach-tegra/Kconfig
arch/powerpc/cpu/ppc4xx/Kconfig
arch/sandbox/Kconfig
arch/x86/Kconfig
board/amcc/canyonlands/Kconfig
board/compulab/cm_t335/Kconfig
board/gumstix/pepper/Kconfig
board/isee/igep0033/Kconfig
board/phytec/pcm051/Kconfig
board/samsung/goni/Kconfig
board/samsung/smdkc100/Kconfig
board/silica/pengwyn/Kconfig
board/ti/am335x/Kconfig
configs/am335x_boneblack_vboot_defconfig
configs/am3517_crane_defconfig
configs/am3517_evm_defconfig
configs/cm_t3517_defconfig
configs/cm_t35_defconfig
configs/devkit8000_defconfig
configs/dig297_defconfig
configs/eco5pk_defconfig
configs/mcx_defconfig
configs/mt_ventoux_defconfig
configs/nokia_rx51_defconfig
configs/omap3_beagle_defconfig
configs/omap3_evm_defconfig
configs/omap3_evm_quick_mmc_defconfig
configs/omap3_evm_quick_nand_defconfig
configs/omap3_ha_defconfig
configs/omap3_logic_defconfig
configs/omap3_mvblx_defconfig
configs/omap3_pandora_defconfig
configs/omap3_sdp3430_defconfig
configs/sandbox_defconfig
configs/tao3530_defconfig
configs/tricorder_defconfig
configs/tricorder_flash_defconfig
configs/twister_defconfig