From mboxrd@z Thu Jan  1 00:00:00 1970
From: Prabhakar Kushwaha <prabhakar@freescale.com>
Date: Wed, 2 Apr 2014 09:15:43 +0530
Subject: [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL
In-Reply-To: <1396389922.32034.23.camel@snotra.buserror.net>
References: <1396260309-2717-1-git-send-email-prabhakar@freescale.com>
	<1396389922.32034.23.camel@snotra.buserror.net>
Message-ID: <533B87E7.6020502@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de


On 4/2/2014 3:35 AM, Scott Wood wrote:
> On Mon, 2014-03-31 at 15:35 +0530, Prabhakar Kushwaha wrote:
>> Objective of this target to have concatenate binary having
>>   - SPL binary in PBL command format
>>   - U-boot binary
>>
>> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
>> ---
>>   Makefile |   19 +++++++++++++++++++
>>   README   |    4 ++++
>>   2 files changed, 23 insertions(+)
>>
>> diff --git a/Makefile b/Makefile
>> index e5f5a8c..e7a0838 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -699,7 +699,11 @@ ALL-y += u-boot.srec u-boot.bin System.map
>>   
>>   ALL-$(CONFIG_NAND_U_BOOT) += u-boot-nand.bin
>>   ALL-$(CONFIG_ONENAND_U_BOOT) += u-boot-onenand.bin
>> +ifeq ($(CONFIG_RAMBOOT_SPLPBL),y)
>> +ALL-$(CONFIG_RAMBOOT_PBL) += u-boot-with-spl-pbl.bin
>> +else
>>   ALL-$(CONFIG_RAMBOOT_PBL) += u-boot.pbl
>> +endif
>>   ALL-$(CONFIG_SPL) += spl/u-boot-spl.bin
>>   ALL-$(CONFIG_SPL_FRAMEWORK) += u-boot.img
>>   ALL-$(CONFIG_TPL) += tpl/u-boot-tpl.bin
>> @@ -882,6 +886,21 @@ endif
>>   u-boot-img.bin: spl/u-boot-spl.bin u-boot.img FORCE
>>   	$(call if_changed,cat)
>>   
>> +#Add a target to create boot binary having SPL binary in PBI format
>> +#concatenated with u-boot binary. It is need by PowerPC SoC having
>> +#internal SRAM <= 512KB.
>> +MKIMAGEFLAGS_u-boot-spl.pbl = -n $(srctree)/$(CONFIG_SYS_FSL_PBL_RCW:"%"=%) \
>> +		-R $(srctree)/$(CONFIG_SYS_FSL_PBL_PBI:"%"=%) -T pblimage
>> +
>> +spl/u-boot-spl.pbl: spl/u-boot-spl.bin FORCE
>> +	$(call if_changed,mkimage)
>> +
>> +OBJCOPYFLAGS_u-boot-with-spl-pbl.bin = -I binary -O binary --pad-to=$(CONFIG_SPL_PAD_TO) \
>> +			  --gap-fill=0xff
>> +
>> +u-boot-with-spl-pbl.bin: spl/u-boot-spl.pbl u-boot.bin FORCE
>> +	$(call if_changed,pad_cat)
>> +
>>   # PPC4xx needs the SPL at the end of the image, since the reset vector
>>   # is located at 0xfffffffc. So we can't use the "u-boot-img.bin" target
>>   # and need to introduce a new build target with the full blown U-Boot
>> diff --git a/README b/README
>> index d8d5f60..7690fdb 100644
>> --- a/README
>> +++ b/README
>> @@ -486,6 +486,10 @@ The following options need to be configured:
>>   		PBI commands can be used to configure SoC before it starts the execution.
>>   		Please refer doc/README.pblimage for more details
>>   
>> +		CONFIG_RAMBOOT_SPLPBL
>> +		It adds a target to create boot binary having SPL binary in PBI format
>> +		concatenated with u-boot binary.
> What does this have to do with RAMBOOT?
>
>
he Image which is required is same as RAMBOOT image. Only thing we don't 
want to have binary in PBI format.
We want to have following kind of binary:
    SPL binary in PBI format + RAMBoot binary.

Regards,
Prabhakar