[U-Boot] [PATCH 10/13 v3] Makefile: Add Kconfig option CONFIG_SPL_IMAGE to select the SPL binary

[Stefan Roese]

On 03.04.19 13:08, Eugen.Hristev at microchip.com wrote:
> 
> 
> On 02.04.2019 11:57, Stefan Roese wrote:
> 
>>
>> This patch adds the CONFIG_SPL_IMAGE option to select the SPL image that
>> shall be used to generate the combined SPL + U-Boot image. The default
>> value is the current value "spl/u-boot-spl.bin".
>>
>> This patch also sets CONFIG_SPL_IMAGE to "spl/boot.bin" for AT91 targets
>> which use SPL NAND support (boot from NAND). For these build targets the
>> combined image "u-boot-with-spl.bin" is now automatically generated and
>> can be programmed into NAND as one single image (vs. SPL image and U-Boot
>> as 2 separate images).
>>
>> Signed-off-by: Stefan Roese <sr at denx.de>
>> Cc: Heiko Schocher <hs at denx.de>
>> Cc: Andreas Bießmann <andreas at biessmann.org>
>> Cc: Eugen Hristev <eugen.hristev at microchip.com>
>> Reviewed-by: Heiko Schocher <hs at denx.de>
>> Tested on the taurus board:
>> Tested-by: Heiko Schocher <hs at denx.de>
>> ---
>> v3:
>> - No change
>>
>> v2:
>> - No change
>>
>>    Kconfig  | 10 ++++++++++
>>    Makefile |  4 +++-
>>    2 files changed, 13 insertions(+), 1 deletion(-)
>>
>> diff --git a/Kconfig b/Kconfig
>> index 305b265ed7..7c2b86f1f3 100644
>> --- a/Kconfig
>> +++ b/Kconfig
>> @@ -224,6 +224,15 @@ config BUILD_ROM
>>    	  which are not shipped in the U-Boot source tree.
>>    	  Please, see doc/README.x86 for details.
>>    
>> +config SPL_IMAGE
>> +	string "SPL image filename that is generated"
>> +	default "spl/boot.bin" if ARCH_AT91 && SPL_NAND_SUPPORT
>> +	default "spl/u-boot-spl.bin"
>> +	help
>> +	  The SPL image filename that is generated by the build process.
>> +	  This image might be used to generated a combined image with
>> +	  SPL and main U-Boot proper as well.
>> +
> 
> Hi Stefan,
> 
> If I try to just use menuconfig and change this value to something else,
> build fails. The purpose of this Kconfig is to name the SPL filename as
> per the config's desire ?
> 
> Or in fact is "which binary file to use to make the Combined SPL+U-BOOT
> mega image" ?

Its not meant to change the SPL image filename to an arbitrary name.
Its meant to select one of the already supported values of the build
system. I should probably re-phrase the Kconfig text to better reflect
this.

Thanks,
Stefan