u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL
@ 2014-03-31 10:05 Prabhakar Kushwaha
  2014-04-01 22:05 ` Scott Wood
  0 siblings, 1 reply; 4+ messages in thread
From: Prabhakar Kushwaha @ 2014-03-31 10:05 UTC (permalink / raw)
  To: u-boot

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@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.
+
 		CONFIG_SYS_FSL_DDR_BE
 		Defines the DDR controller register space as Big Endian
 
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL
  2014-03-31 10:05 [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL Prabhakar Kushwaha
@ 2014-04-01 22:05 ` Scott Wood
  2014-04-02  3:45   ` Prabhakar Kushwaha
  0 siblings, 1 reply; 4+ messages in thread
From: Scott Wood @ 2014-04-01 22:05 UTC (permalink / raw)
  To: u-boot

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?

-Scott

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL
  2014-04-01 22:05 ` Scott Wood
@ 2014-04-02  3:45   ` Prabhakar Kushwaha
  2014-04-04 21:49     ` Scott Wood
  0 siblings, 1 reply; 4+ messages in thread
From: Prabhakar Kushwaha @ 2014-04-02  3:45 UTC (permalink / raw)
  To: u-boot


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL
  2014-04-02  3:45   ` Prabhakar Kushwaha
@ 2014-04-04 21:49     ` Scott Wood
  0 siblings, 0 replies; 4+ messages in thread
From: Scott Wood @ 2014-04-04 21:49 UTC (permalink / raw)
  To: u-boot

On Wed, 2014-04-02 at 09:15 +0530, Prabhakar Kushwaha wrote:
> 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.

I'd call it CONFIG_SPL_FSL_PBL or similar.  It's not RAM boot.

-Scott

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-04-04 21:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-31 10:05 [U-Boot] [PATCH 8/10] Makefile: Add support of RAMBOOT_SPLPBL Prabhakar Kushwaha
2014-04-01 22:05 ` Scott Wood
2014-04-02  3:45   ` Prabhakar Kushwaha
2014-04-04 21:49     ` Scott Wood

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).