[U-Boot] [PATCH v3 1/2] spl: Convert CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR to Kconfig

[Tom Rini]

On Tue, Nov 01, 2016 at 03:29:09PM +0200, Sam Protsenko wrote:

> Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
> ---
> Changes in v3:
>   - PATCH 1/2: add USE_SECTOR to missed boards (reported by buildman):
>                - Merrii_A80_Optimus
>                - Cubieboard4
>                - imx6qdl_icore_mmc
> Changes in v2:
>   - PATCH 1/2: merge "SPL dependency" patch
>   - PATCH 1/2: add defaults for common architectures/targets
[snip]
>  311 files changed, 331 insertions(+), 89 deletions(-)

Getting better, but..

[snip]
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index bb99f1f..1ecc6ff 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -70,6 +70,36 @@ config SPL_DISPLAY_PRINT
>  	  banner ("U-Boot SPL ..."). This function should be provided by
>  	  the board.
>  
> +config SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> +	bool "MMC raw mode: by sector"
> +	depends on SPL
> +	default n
> +	help
> +	  Use sector number for specifying U-Boot location on MMC/SD in
> +	  raw mode.

This should be default y if ARCH_SUNXI and perhaps a few others from the
list where you have defaults for the sector.  That said..

> +config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
> +	hex "Address on the MMC to load U-Boot from"
> +	depends on SPL && SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> +	default 0x50 if ARCH_SUNXI
> +	default 0x75 if ARCH_DAVINCI
> +	default 0x80 if ARCH_UNIPHIER
> +	default 0x8a if ARCH_MX6
> +	default 0xe8 if TARGET_LS1021AQDS || TARGET_LS1021ATWR
> +	default 0xf0 if TARGET_LS1043AQDS || TARGET_LS1043ARDB
> +	default 0x100 if ARCH_ROCKCHIP || TARGET_WOODBURN_SD
> +	default 0x110 if TARGET_LS1046AQDS || TARGET_LS1046ARDB
> +	default 0x140 if ARCH_MVEBU
> +	default 0x200 if ARCH_SOCFPGA || ARCH_AT91
> +	default 0x300 if ARCH_ZYNQ || ARCH_KEYSTONE || OMAP34XX || OMAP44XX || \
> +		         OMAP54XX || AM33XX || AM43XX || TARGET_BRPPT1 || \
> +		         TARGET_BRXRE1 || TARGET_DRACO || TARGET_ETAMIN || \
> +		         TARGET_PXM2 || TARGET_RASTABAN || TARGET_RUT || \
> +		         TARGET_THUBAN || TARGET_TI814X_EVM || TARGET_TI816X_EVM

All of the cases of TARGET_... should be done in the config file as they
are target-dependent.  I think a few of those even will be caught by
AM33XX as they just didn't use ti_armv7_common.h previously.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161101/800e1990/attachment.sig>