[U-Boot] [PATCH 2/4] Convert CONFIG_BOARD_TYPES to Kconfig
Alexey Brodkin
alexey.brodkin at synopsys.com
Mon Dec 10 07:37:14 UTC 2018
Hi Simon,
On Sun, 2018-12-09 at 18:44 -0700, Simon Glass wrote:
> This converts the following to Kconfig:
> CONFIG_BOARD_TYPES
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> arch/arm/mach-exynos/Kconfig | 8 ++++++++
> configs/odroid-xu3_defconfig | 2 +-
> configs/odroid_defconfig | 2 +-
> include/configs/axs10x.h | 1 -
> include/configs/hsdk.h | 1 -
Please note ARC boards have nothing to do with Samsung Exynos.
They are based on ARC cores and "arch/arm/mach-exynos/Kconfig"
makes no sense for them.
Moreover I'm not sure why CONFIG_BOARD_TYPES is an
Exynos-specific option. That said why don't we move CONFIG_BOARD_TYPES
to the generic Kconfig?
> include/configs/odroid.h | 1 -
> include/configs/odroid_xu3.h | 1 -
> scripts/config_whitelist.txt | 1 -
> 8 files changed, 10 insertions(+), 7 deletions(-)
[snip]
> diff --git a/include/configs/axs10x.h b/include/configs/axs10x.h
> index 1b2966fd612..a1b9e7b21d9 100644
> --- a/include/configs/axs10x.h
> +++ b/include/configs/axs10x.h
> @@ -34,7 +34,6 @@
> /*
> * This board might be of different versions so handle it
> */
There's no reason to keep above comment if "#define" below is gone.
> -#define CONFIG_BOARD_TYPES
>
> /*
> * NAND Flash configuration
> diff --git a/include/configs/hsdk.h b/include/configs/hsdk.h
> index cdf4fddd940..b1af1271777 100644
> --- a/include/configs/hsdk.h
> +++ b/include/configs/hsdk.h
> @@ -35,7 +35,6 @@
> /*
> * This board might be of different versions so handle it
> */
Ditto. But then it turned out in case of HSDK CONFIG_BOARD_TYPES
is just a copy-paste from AXS10x and "gd->board_type" is not used.
That said this hunk is applicable but I may do it myself separately.
> -#define CONFIG_BOARD_TYPES
>
> /*
> * UART configuration
-Alexey
More information about the U-Boot
mailing list