[U-Boot] [PATCH v2 10/10] sunxi: Convert CONS_INDEX to Kconfig

[Maxime Ripard]

Hi,

On Wed, Mar 29, 2017 at 09:26:37AM +0200, Mylène Josserand wrote:
> Convert the CONS_INDEX configuration to Kconfig.
> Update sunxi's defconfigs to remove SYS_EXTRA_OPTIONS variable not
> needed anymore.
> Default value is 1 except for sun5i (equals 2) and sun8i (equals 5).
> 
> Signed-off-by: Mylène Josserand <mylene.josserand at free-electrons.com>
> ---
>  board/sunxi/Kconfig                      | 9 +++++++++
>  configs/A13-OLinuXinoM_defconfig         | 1 -
>  configs/A13-OLinuXino_defconfig          | 1 -
>  configs/Ampe_A76_defconfig               | 1 -
>  configs/CHIP_defconfig                   | 1 -
>  configs/Empire_electronix_d709_defconfig | 1 -
>  configs/Empire_electronix_m712_defconfig | 1 -
>  configs/difrnce_dit4350_defconfig        | 1 -
>  configs/ga10h_v1_1_defconfig             | 1 -
>  configs/gt90h_v4_defconfig               | 1 -
>  configs/iNet_D978_rev2_defconfig         | 1 -
>  configs/inet86dz_defconfig               | 1 -
>  configs/inet98v_rev2_defconfig           | 1 -
>  configs/polaroid_mid2407pxe03_defconfig  | 1 -
>  configs/polaroid_mid2809pxe04_defconfig  | 1 -
>  configs/q8_a13_tablet_defconfig          | 1 -
>  configs/q8_a23_tablet_800x480_defconfig  | 1 -
>  configs/q8_a33_tablet_1024x600_defconfig | 1 -
>  configs/q8_a33_tablet_800x480_defconfig  | 1 -
>  include/configs/sunxi-common.h           | 4 ----
>  20 files changed, 9 insertions(+), 22 deletions(-)


Did you rebase on the current sunxi tree recently? There's at least
two boards (CHIP Pro and A23 EVB) that are unfortunately missing from
that least, and whose compilation now will cause a warning :/

The CHIP Pro also has a bunch of other (new) options in
SYS_EXTRA_OPTIONS, but you don't have to convert them if you don't
want to.

> diff --git a/board/sunxi/Kconfig b/board/sunxi/Kconfig
> index 9963b6b5d8..706c62d233 100644
> --- a/board/sunxi/Kconfig
> +++ b/board/sunxi/Kconfig
> @@ -157,6 +157,15 @@ config ARM_BOOT_HOOK_RMR
>  	This allows both the SPL and the U-Boot proper to be entered in
>  	either mode and switch to AArch64 if needed.
>  
> +config CONS_INDEX
> +	int "UART used for console"
> +	default 2 if MACH_SUN5I
> +	default 5 if MACH_SUN8I
> +	default 1
> +	help
> +	  Configures the console index for Allwinner SoC. 2 for SUN5I, 5 for SUN8I
> +	  and 1 otherwise.

This is a generic option that is used across a number of platforms,
perhaps this would be better suited to be in drivers/serial (or some
place else?)

Once fixed,
Acked-by: Maxime Ripard <maxime.ripard at free-electrons.com>

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170329/28b8ca94/attachment.sig>