[U-Boot] [PATCH] disk: convert to Kconfig

[Tom Rini]

On Wed, Nov 30, 2016 at 11:39:09AM +0100, Patrick Delaunay wrote:

> This converts the following to Kconfig:
>        CONFIG_PARTITIONS
>        CONFIG_MAC_PARTITION
>        CONFIG_DOS_PARTITION
>        CONFIG_ISO_PARTITION
>        CONFIG_AMIGA_PARTITION
>        CONFIG_EFI_PARTITION
>        CONFIG_PARTITION_UUIDS
>        CONFIG_PARTITION_TYPE_GUID
> 
> Specific SPL config are added to reduce SPL size
> for some target and solve compilation issues
> 
>        CONFIG_SPL_MAC_PARTITION
>        CONFIG_SPL_DOS_PARTITION
>        CONFIG_SPL_ISO_PARTITION
>        CONFIG_SPL_AMIGA_PARTITION
>        CONFIG_SPL_EFI_PARTITION
> 
> Associated test are updated to the correct CONFIG (SPL or not)
> by using macro CONFIG_IS_ENABLED
> 
> To update the U-Boot code I use :
> 
> 1- update defconfig for new SPL option
>    ./tools/moveconfig.py --spl --commit \
>        PARTITIONS MAC_PARTITION DOS_PARTITION \
>        ISO_PARTITION AMIGA_PARTITION EFI_PARTITION \
>        PARTITION_UUIDS PARTITION_TYPE_GUID
> 
>    then replace CONFIG by CONFIG_SPL in defconfig
>    sed -i "s/CONFIG_\(DOS\|EFI\|ISO\|MAC\|AMIGA\)_PARTITION/CONFIG_SPL_\1_PARTITION/" \
>        configs/*
> 
> 2- update defconfig for existing option
> ./tools/moveconfig.py --commit --yes \
>        PARTITIONS MAC_PARTITION DOS_PARTITION \
>        ISO_PARTITION AMIGA_PARTITION EFI_PARTITION \
>        PARTITION_UUIDS PARTITION_TYPE_GUID
> 
> => squash the 2 modifications
> 
> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> Signed-off-by: Patrick Delaunay <patrick.delaunay73 at gmail.com>
> ---
> move disk to Kconfig as proposed by Simon Glass in thread
> [Resend RFC PATCH v1 1/3] add support of GPT partitioning over MTD
> 
> NB: I create _SPL_ config because I have issue
>     with size of SPL for some platform
> 
> => list of supported partition type is not the same between SPL
>    and U-Boot, that allow size reduction for SPL build
>    (CONFIG are under compilation flag CONFIG_SPL_BUILD in .h)
> 
> I compile this patch on v2016.11 without issue linked to this code
> on all platform and this commit is rebased on master
[snip]
>  1203 files changed, 2253 insertions(+), 798 deletions(-)

OK, we've got a few things to fix in here.

> diff --git a/Kconfig b/Kconfig
> index 529858a..46c373e 100644
> --- a/Kconfig
> +++ b/Kconfig
[snip]
> diff --git a/configs/A10-OLinuXino-Lime_defconfig b/configs/A10-OLinuXino-Lime_defconfig
> index bb7eaf8..50436a7 100644
> --- a/configs/A10-OLinuXino-Lime_defconfig
> +++ b/configs/A10-OLinuXino-Lime_defconfig
> @@ -7,8 +7,8 @@ CONFIG_DRAM_EMR1=4
>  CONFIG_SYS_CLK_FREQ=912000000
>  CONFIG_MMC0_CD_PIN="PH1"
>  CONFIG_DEFAULT_DEVICE_TREE="sun4i-a10-olinuxino-lime"
> -# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
>  CONFIG_AHCI=y
> +# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
>  CONFIG_SYS_EXTRA_OPTIONS="SUNXI_EMAC,SATAPWR=SUNXI_GPC(3)"
>  CONFIG_SPL=y
>  # CONFIG_CMD_IMLS is not set

There's a lot of changes like this, just drop them please.

> diff --git a/disk/Kconfig b/disk/Kconfig
> new file mode 100644
> index 0000000..57da657
> --- /dev/null
> +++ b/disk/Kconfig
[snip]
> +config DOS_PARTITION
> +	bool "Enable MS Dos partition table"
> +	depends on PARTITIONS
> +	help
> +	  traditional on the Intel architecture, USB sticks, etc.
> +
> +config SPL_DOS_PARTITION
> +	bool "Enable MS Dos partition table for SPL"
> +	depends on SPL && PARTITIONS

These should both be default y.  Doing both of these will make the rest
of the patch much more reviewable, 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/20161130/cacc85cf/attachment.sig>