[PATCH] recipe: u-boot: update recipe for more images

[PATCH] recipe: u-boot: update recipe for more images

[Murali Karicheri]

On Keystone EVM, u-boot build needs to create following additional
binaries and images
 - u-boot-spl.bin
	- This for first stage boot from SPI NOR flash
 - u-boot.img
	- This is the second stage u-boot image in uImage format.
 - u-boot-spi.gph
	- This combines the above two images into one image so that
	  it is easy to program on the device.

Signed-off-by: Murali Karicheri <m-karicheri2@ti.com>
---
 SRC_URI and SRCREV will be replaced before pushing this recipe to repo.
 recipes-bsp/u-boot/u-boot_2013.01.bb |   64 ++++++++++++++++++++++++++++++++--
 1 file changed, 62 insertions(+), 2 deletions(-)

diff --git a/recipes-bsp/u-boot/u-boot_2013.01.bb b/recipes-bsp/u-boot/u-boot_2013.01.bb
index b9e6daf..07e8d6e 100644
--- a/recipes-bsp/u-boot/u-boot_2013.01.bb
+++ b/recipes-bsp/u-boot/u-boot_2013.01.bb
@@ -10,14 +10,74 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
 
 PR = "r2+gitr${SRCPV}"
 
-SRC_URI = "git://arago-project.org/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
+#SRC_URI = "git://arago-project.org/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
+SRC_URI = "git://gtgit01.gt.design.ti.com/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
 
 BRANCH = "master"
 
-SRCREV = "DEV.MCSDK-03.00.00.07"
+#SRCREV = "DEV.MCSDK-03.00.00.08"
+SRCREV = "327e9f591d6530650609915c7c35c874bfc25550"
 
 EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}"'
 
 S = "${WORKDIR}/git"
 
 UBOOT_SUFFIX = "bin"
+
+UBOOT_MAKE_TARGET = "u-boot-spi.gph"
+# SPI NOR Flash binaries
+UBOOT_SPI_SPL_BINARY = "u-boot-spl.bin"
+UBOOT_SPI_BINARY = "u-boot.img"
+UBOOT_SPI_GPH_BINARY = "u-boot-spi.gph"
+# SPI NOR Flash deployed images
+UBOOT_SPI_SPL_IMAGE = "u-boot-spl-${MACHINE}-${PV}-${PR}.bin"
+UBOOT_SPI_SPL_SYMLINK = "u-boot-spl-${MACHINE}.bin"
+UBOOT_SPI_IMAGE = "u-boot-${MACHINE}-${PV}-${PR}.img"
+UBOOT_SPI_SYMLINK = "u-boot-${MACHINE}.img"
+UBOOT_SPI_GPH_IMAGE = "u-boot-spi-${MACHINE}-${PV}-${PR}.gph"
+UBOOT_SPI_GPH_SYMLINK = "u-boot-spi-${MACHINE}.gph"
+
+do_configure () {
+	oe_runmake ${UBOOT_MACHINE}
+}
+
+do_compile () {
+	unset LDFLAGS
+	unset CFLAGS
+	unset CPPFLAGS
+	oe_runmake ${UBOOT_MAKE_TARGET}
+}
+
+do_install () {
+	install -d ${D}/boot
+	install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
+	install ${S}/spl/${UBOOT_SPI_SPL_BINARY} ${D}/boot/${UBOOT_SPI_SPL_IMAGE}
+	install ${S}/${UBOOT_SPI_BINARY} ${D}/boot/${UBOOT_SPI_IMAGE}
+	install ${S}/${UBOOT_SPI_GPH_BINARY} ${D}/boot/${UBOOT_SPI_GPH_IMAGE}
+	ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
+	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${D}/boot/${UBOOT_SPI_SPL_BINARY}
+	ln -sf ${UBOOT_SPI_IMAGE} ${D}/boot/${UBOOT_SPI_BINARY}
+	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${D}/boot/${UBOOT_SPI_GPH_BINARY}
+}
+
+do_deploy () {
+	install -d ${DEPLOY_DIR_IMAGE}
+	install ${S}/${UBOOT_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_IMAGE}
+	install ${S}/spl/${UBOOT_SPI_SPL_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_SPL_IMAGE}
+	install ${S}/${UBOOT_SPI_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_IMAGE}
+	install ${S}/${UBOOT_SPI_GPH_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_GPH_IMAGE}
+
+	cd ${DEPLOY_DIR_IMAGE}
+	rm -f ${UBOOT_BINARY} ${UBOOT_SYMLINK}
+	ln -sf ${UBOOT_IMAGE} ${UBOOT_SYMLINK}
+	ln -sf ${UBOOT_IMAGE} ${UBOOT_BINARY}
+	rm -f ${UBOOT_SPI_SPL_BINARY} ${UBOOT_SPI_SPL_SYMLINK}
+	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${UBOOT_SPI_SPL_SYMLINK}
+	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${UBOOT_SPI_SPL_BINARY}
+	rm -f ${UBOOT_SPI_BINARY} ${UBOOT_SPI_SYMLINK}
+	ln -sf ${UBOOT_SPI_IMAGE} ${UBOOT_SPI_SYMLINK}
+	ln -sf ${UBOOT_SPI_IMAGE} ${UBOOT_SPI_BINARY}
+	rm -f ${UBOOT_SPI_GPH_BINARY} ${UBOOT_SPI_GPH_SYMLINK}
+	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${UBOOT_SPI_GPH_SYMLINK}
+	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${UBOOT_SPI_GPH_BINARY}
+}
-- 
1.7.9.5

Re: [PATCH] recipe: u-boot: update recipe for more images

[Murali Karicheri]

Denys,

I understand that this recipe is  not complete as u-boot_2013.01.bb may 
be shared by other machines and this change is not applicable.
Do I need to add this to some other file? Please advise.

Meanwhile for the release this should work as this is going into mcsdk 
meta-ti repo. But to push this to meta-ti, the above change is needed 
and hence need your input.

Murali
On 2/26/2013 1:38 PM, Murali Karicheri wrote:
> On Keystone EVM, u-boot build needs to create following additional
> binaries and images
>   - u-boot-spl.bin
> 	- This for first stage boot from SPI NOR flash
>   - u-boot.img
> 	- This is the second stage u-boot image in uImage format.
>   - u-boot-spi.gph
> 	- This combines the above two images into one image so that
> 	  it is easy to program on the device.
>
> Signed-off-by: Murali Karicheri <m-karicheri2@ti.com>
> ---
>   SRC_URI and SRCREV will be replaced before pushing this recipe to repo.
>   recipes-bsp/u-boot/u-boot_2013.01.bb |   64 ++++++++++++++++++++++++++++++++--
>   1 file changed, 62 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-bsp/u-boot/u-boot_2013.01.bb b/recipes-bsp/u-boot/u-boot_2013.01.bb
> index b9e6daf..07e8d6e 100644
> --- a/recipes-bsp/u-boot/u-boot_2013.01.bb
> +++ b/recipes-bsp/u-boot/u-boot_2013.01.bb
> @@ -10,14 +10,74 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
>   
>   PR = "r2+gitr${SRCPV}"
>   
> -SRC_URI = "git://arago-project.org/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
> +#SRC_URI = "git://arago-project.org/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
> +SRC_URI = "git://gtgit01.gt.design.ti.com/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
>   
>   BRANCH = "master"
>   
> -SRCREV = "DEV.MCSDK-03.00.00.07"
> +#SRCREV = "DEV.MCSDK-03.00.00.08"
> +SRCREV = "327e9f591d6530650609915c7c35c874bfc25550"
>   
>   EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}"'
>   
>   S = "${WORKDIR}/git"
>   
>   UBOOT_SUFFIX = "bin"
> +
> +UBOOT_MAKE_TARGET = "u-boot-spi.gph"
> +# SPI NOR Flash binaries
> +UBOOT_SPI_SPL_BINARY = "u-boot-spl.bin"
> +UBOOT_SPI_BINARY = "u-boot.img"
> +UBOOT_SPI_GPH_BINARY = "u-boot-spi.gph"
> +# SPI NOR Flash deployed images
> +UBOOT_SPI_SPL_IMAGE = "u-boot-spl-${MACHINE}-${PV}-${PR}.bin"
> +UBOOT_SPI_SPL_SYMLINK = "u-boot-spl-${MACHINE}.bin"
> +UBOOT_SPI_IMAGE = "u-boot-${MACHINE}-${PV}-${PR}.img"
> +UBOOT_SPI_SYMLINK = "u-boot-${MACHINE}.img"
> +UBOOT_SPI_GPH_IMAGE = "u-boot-spi-${MACHINE}-${PV}-${PR}.gph"
> +UBOOT_SPI_GPH_SYMLINK = "u-boot-spi-${MACHINE}.gph"
> +
> +do_configure () {
> +	oe_runmake ${UBOOT_MACHINE}
> +}
> +
> +do_compile () {
> +	unset LDFLAGS
> +	unset CFLAGS
> +	unset CPPFLAGS
> +	oe_runmake ${UBOOT_MAKE_TARGET}
> +}
> +
> +do_install () {
> +	install -d ${D}/boot
> +	install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
> +	install ${S}/spl/${UBOOT_SPI_SPL_BINARY} ${D}/boot/${UBOOT_SPI_SPL_IMAGE}
> +	install ${S}/${UBOOT_SPI_BINARY} ${D}/boot/${UBOOT_SPI_IMAGE}
> +	install ${S}/${UBOOT_SPI_GPH_BINARY} ${D}/boot/${UBOOT_SPI_GPH_IMAGE}
> +	ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
> +	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${D}/boot/${UBOOT_SPI_SPL_BINARY}
> +	ln -sf ${UBOOT_SPI_IMAGE} ${D}/boot/${UBOOT_SPI_BINARY}
> +	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${D}/boot/${UBOOT_SPI_GPH_BINARY}
> +}
> +
> +do_deploy () {
> +	install -d ${DEPLOY_DIR_IMAGE}
> +	install ${S}/${UBOOT_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_IMAGE}
> +	install ${S}/spl/${UBOOT_SPI_SPL_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_SPL_IMAGE}
> +	install ${S}/${UBOOT_SPI_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_IMAGE}
> +	install ${S}/${UBOOT_SPI_GPH_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_GPH_IMAGE}
> +
> +	cd ${DEPLOY_DIR_IMAGE}
> +	rm -f ${UBOOT_BINARY} ${UBOOT_SYMLINK}
> +	ln -sf ${UBOOT_IMAGE} ${UBOOT_SYMLINK}
> +	ln -sf ${UBOOT_IMAGE} ${UBOOT_BINARY}
> +	rm -f ${UBOOT_SPI_SPL_BINARY} ${UBOOT_SPI_SPL_SYMLINK}
> +	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${UBOOT_SPI_SPL_SYMLINK}
> +	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${UBOOT_SPI_SPL_BINARY}
> +	rm -f ${UBOOT_SPI_BINARY} ${UBOOT_SPI_SYMLINK}
> +	ln -sf ${UBOOT_SPI_IMAGE} ${UBOOT_SPI_SYMLINK}
> +	ln -sf ${UBOOT_SPI_IMAGE} ${UBOOT_SPI_BINARY}
> +	rm -f ${UBOOT_SPI_GPH_BINARY} ${UBOOT_SPI_GPH_SYMLINK}
> +	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${UBOOT_SPI_GPH_SYMLINK}
> +	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${UBOOT_SPI_GPH_BINARY}
> +}

Re: [PATCH] recipe: u-boot: update recipe for more images

[Denys Dmytriyenko]

On Tue, Feb 26, 2013 at 01:42:31PM -0500, Murali Karicheri wrote:
> Denys,
> 
> I understand that this recipe is  not complete as u-boot_2013.01.bb
> may be shared by other machines and this change is not applicable.
> Do I need to add this to some other file? Please advise.
> 
> Meanwhile for the release this should work as this is going into
> mcsdk meta-ti repo. But to push this to meta-ti, the above change is
> needed and hence need your input.

Murali,

I know Carlos has been working on adding SPL support to u-boot already:

http://arago-project.org/pipermail/meta-arago/2013-February/000937.html

Can you sync up with him and come up with a unified solution for both SPL and 
GPH images that works for everybody and I'd be happy to push it to meta-ti.

Also, I'd like to hear Tom's opinion on this, as current U-Boot maintainer - 
if there is a better approach to this exists or planned in mainline.

Thanks.

-- 
Denys


> On 2/26/2013 1:38 PM, Murali Karicheri wrote:
> >On Keystone EVM, u-boot build needs to create following additional
> >binaries and images
> >  - u-boot-spl.bin
> >	- This for first stage boot from SPI NOR flash
> >  - u-boot.img
> >	- This is the second stage u-boot image in uImage format.
> >  - u-boot-spi.gph
> >	- This combines the above two images into one image so that
> >	  it is easy to program on the device.
> >
> >Signed-off-by: Murali Karicheri <m-karicheri2@ti.com>
> >---
> >  SRC_URI and SRCREV will be replaced before pushing this recipe to repo.
> >  recipes-bsp/u-boot/u-boot_2013.01.bb |   64 ++++++++++++++++++++++++++++++++--
> >  1 file changed, 62 insertions(+), 2 deletions(-)
> >
> >diff --git a/recipes-bsp/u-boot/u-boot_2013.01.bb b/recipes-bsp/u-boot/u-boot_2013.01.bb
> >index b9e6daf..07e8d6e 100644
> >--- a/recipes-bsp/u-boot/u-boot_2013.01.bb
> >+++ b/recipes-bsp/u-boot/u-boot_2013.01.bb
> >@@ -10,14 +10,74 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
> >  PR = "r2+gitr${SRCPV}"
> >-SRC_URI = "git://arago-project.org/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
> >+#SRC_URI = "git://arago-project.org/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
> >+SRC_URI = "git://gtgit01.gt.design.ti.com/git/projects/u-boot-keystone.git;protocol=git;branch=${BRANCH}"
> >  BRANCH = "master"
> >-SRCREV = "DEV.MCSDK-03.00.00.07"
> >+#SRCREV = "DEV.MCSDK-03.00.00.08"
> >+SRCREV = "327e9f591d6530650609915c7c35c874bfc25550"
> >  EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}"'
> >  S = "${WORKDIR}/git"
> >  UBOOT_SUFFIX = "bin"
> >+
> >+UBOOT_MAKE_TARGET = "u-boot-spi.gph"
> >+# SPI NOR Flash binaries
> >+UBOOT_SPI_SPL_BINARY = "u-boot-spl.bin"
> >+UBOOT_SPI_BINARY = "u-boot.img"
> >+UBOOT_SPI_GPH_BINARY = "u-boot-spi.gph"
> >+# SPI NOR Flash deployed images
> >+UBOOT_SPI_SPL_IMAGE = "u-boot-spl-${MACHINE}-${PV}-${PR}.bin"
> >+UBOOT_SPI_SPL_SYMLINK = "u-boot-spl-${MACHINE}.bin"
> >+UBOOT_SPI_IMAGE = "u-boot-${MACHINE}-${PV}-${PR}.img"
> >+UBOOT_SPI_SYMLINK = "u-boot-${MACHINE}.img"
> >+UBOOT_SPI_GPH_IMAGE = "u-boot-spi-${MACHINE}-${PV}-${PR}.gph"
> >+UBOOT_SPI_GPH_SYMLINK = "u-boot-spi-${MACHINE}.gph"
> >+
> >+do_configure () {
> >+	oe_runmake ${UBOOT_MACHINE}
> >+}
> >+
> >+do_compile () {
> >+	unset LDFLAGS
> >+	unset CFLAGS
> >+	unset CPPFLAGS
> >+	oe_runmake ${UBOOT_MAKE_TARGET}
> >+}
> >+
> >+do_install () {
> >+	install -d ${D}/boot
> >+	install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
> >+	install ${S}/spl/${UBOOT_SPI_SPL_BINARY} ${D}/boot/${UBOOT_SPI_SPL_IMAGE}
> >+	install ${S}/${UBOOT_SPI_BINARY} ${D}/boot/${UBOOT_SPI_IMAGE}
> >+	install ${S}/${UBOOT_SPI_GPH_BINARY} ${D}/boot/${UBOOT_SPI_GPH_IMAGE}
> >+	ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
> >+	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${D}/boot/${UBOOT_SPI_SPL_BINARY}
> >+	ln -sf ${UBOOT_SPI_IMAGE} ${D}/boot/${UBOOT_SPI_BINARY}
> >+	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${D}/boot/${UBOOT_SPI_GPH_BINARY}
> >+}
> >+
> >+do_deploy () {
> >+	install -d ${DEPLOY_DIR_IMAGE}
> >+	install ${S}/${UBOOT_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_IMAGE}
> >+	install ${S}/spl/${UBOOT_SPI_SPL_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_SPL_IMAGE}
> >+	install ${S}/${UBOOT_SPI_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_IMAGE}
> >+	install ${S}/${UBOOT_SPI_GPH_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_SPI_GPH_IMAGE}
> >+
> >+	cd ${DEPLOY_DIR_IMAGE}
> >+	rm -f ${UBOOT_BINARY} ${UBOOT_SYMLINK}
> >+	ln -sf ${UBOOT_IMAGE} ${UBOOT_SYMLINK}
> >+	ln -sf ${UBOOT_IMAGE} ${UBOOT_BINARY}
> >+	rm -f ${UBOOT_SPI_SPL_BINARY} ${UBOOT_SPI_SPL_SYMLINK}
> >+	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${UBOOT_SPI_SPL_SYMLINK}
> >+	ln -sf ${UBOOT_SPI_SPL_IMAGE} ${UBOOT_SPI_SPL_BINARY}
> >+	rm -f ${UBOOT_SPI_BINARY} ${UBOOT_SPI_SYMLINK}
> >+	ln -sf ${UBOOT_SPI_IMAGE} ${UBOOT_SPI_SYMLINK}
> >+	ln -sf ${UBOOT_SPI_IMAGE} ${UBOOT_SPI_BINARY}
> >+	rm -f ${UBOOT_SPI_GPH_BINARY} ${UBOOT_SPI_GPH_SYMLINK}
> >+	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${UBOOT_SPI_GPH_SYMLINK}
> >+	ln -sf ${UBOOT_SPI_GPH_IMAGE} ${UBOOT_SPI_GPH_BINARY}
> >+}
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
> 

Re: [PATCH] recipe: u-boot: update recipe for more images

[Tom Rini]

On 02/26/2013 02:20 PM, Denys Dmytriyenko wrote:
> On Tue, Feb 26, 2013 at 01:42:31PM -0500, Murali Karicheri wrote:
>> Denys,
>>
>> I understand that this recipe is  not complete as u-boot_2013.01.bb
>> may be shared by other machines and this change is not applicable.
>> Do I need to add this to some other file? Please advise.
>>
>> Meanwhile for the release this should work as this is going into
>> mcsdk meta-ti repo. But to push this to meta-ti, the above change is
>> needed and hence need your input.
> 
> Murali,
> 
> I know Carlos has been working on adding SPL support to u-boot already:
> 
> http://arago-project.org/pipermail/meta-arago/2013-February/000937.html
> 
> Can you sync up with him and come up with a unified solution for both SPL and 
> GPH images that works for everybody and I'd be happy to push it to meta-ti.
> 
> Also, I'd like to hear Tom's opinion on this, as current U-Boot maintainer - 
> if there is a better approach to this exists or planned in mainline.

I haven't seen the keystone patches yet, so I don't know if the GPH
image is "special" in terms of headers, but the build target
u-boot-with-spl.bin is the canonical build target for "put
u-boot-spl.bin and u-boot.bin together".

-- 
Tom

Re: [PATCH] recipe: u-boot: update recipe for more images

[Murali Karicheri]

+ Cyril,

Cyril had worked on adding GPH format for u-boot. Want to hear from him 
why not used the u-boot-with-spl.bin build target.

Murali
On 2/26/2013 2:31 PM, Tom Rini wrote:
> On 02/26/2013 02:20 PM, Denys Dmytriyenko wrote:
>> On Tue, Feb 26, 2013 at 01:42:31PM -0500, Murali Karicheri wrote:
>>> Denys,
>>>
>>> I understand that this recipe is  not complete as u-boot_2013.01.bb
>>> may be shared by other machines and this change is not applicable.
>>> Do I need to add this to some other file? Please advise.
>>>
>>> Meanwhile for the release this should work as this is going into
>>> mcsdk meta-ti repo. But to push this to meta-ti, the above change is
>>> needed and hence need your input.
>> Murali,
>>
>> I know Carlos has been working on adding SPL support to u-boot already:
>>
>> http://arago-project.org/pipermail/meta-arago/2013-February/000937.html
>>
>> Can you sync up with him and come up with a unified solution for both SPL and
>> GPH images that works for everybody and I'd be happy to push it to meta-ti.
>>
>> Also, I'd like to hear Tom's opinion on this, as current U-Boot maintainer -
>> if there is a better approach to this exists or planned in mainline.
> I haven't seen the keystone patches yet, so I don't know if the GPH
> image is "special" in terms of headers, but the build target
> u-boot-with-spl.bin is the canonical build target for "put
> u-boot-spl.bin and u-boot.bin together".
>

Re: [PATCH] recipe: u-boot: update recipe for more images

[Denys Dmytriyenko]

Actually adding Cyril this time.

On Tue, Feb 26, 2013 at 02:53:52PM -0500, Murali Karicheri wrote:
> + Cyril,
> 
> Cyril had worked on adding GPH format for u-boot. Want to hear from
> him why not used the u-boot-with-spl.bin build target.
> 
> Murali
> On 2/26/2013 2:31 PM, Tom Rini wrote:
> >On 02/26/2013 02:20 PM, Denys Dmytriyenko wrote:
> >>On Tue, Feb 26, 2013 at 01:42:31PM -0500, Murali Karicheri wrote:
> >>>Denys,
> >>>
> >>>I understand that this recipe is  not complete as u-boot_2013.01.bb
> >>>may be shared by other machines and this change is not applicable.
> >>>Do I need to add this to some other file? Please advise.
> >>>
> >>>Meanwhile for the release this should work as this is going into
> >>>mcsdk meta-ti repo. But to push this to meta-ti, the above change is
> >>>needed and hence need your input.
> >>Murali,
> >>
> >>I know Carlos has been working on adding SPL support to u-boot already:
> >>
> >>http://arago-project.org/pipermail/meta-arago/2013-February/000937.html
> >>
> >>Can you sync up with him and come up with a unified solution for both SPL and
> >>GPH images that works for everybody and I'd be happy to push it to meta-ti.
> >>
> >>Also, I'd like to hear Tom's opinion on this, as current U-Boot maintainer -
> >>if there is a better approach to this exists or planned in mainline.
> >I haven't seen the keystone patches yet, so I don't know if the GPH
> >image is "special" in terms of headers, but the build target
> >u-boot-with-spl.bin is the canonical build target for "put
> >u-boot-spl.bin and u-boot.bin together".

Re: [PATCH] recipe: u-boot: update recipe for more images

[Denys Dmytriyenko]

On Tue, Feb 26, 2013 at 03:02:49PM -0500, Denys Dmytriyenko wrote:
> Actually adding Cyril this time.

Ah, never mind - Cyril was already copied, but mailman filters out emails that 
are not subscribed...

> On Tue, Feb 26, 2013 at 02:53:52PM -0500, Murali Karicheri wrote:
> > + Cyril,
> > 
> > Cyril had worked on adding GPH format for u-boot. Want to hear from
> > him why not used the u-boot-with-spl.bin build target.
> > 
> > Murali
> > On 2/26/2013 2:31 PM, Tom Rini wrote:
> > >On 02/26/2013 02:20 PM, Denys Dmytriyenko wrote:
> > >>On Tue, Feb 26, 2013 at 01:42:31PM -0500, Murali Karicheri wrote:
> > >>>Denys,
> > >>>
> > >>>I understand that this recipe is  not complete as u-boot_2013.01.bb
> > >>>may be shared by other machines and this change is not applicable.
> > >>>Do I need to add this to some other file? Please advise.
> > >>>
> > >>>Meanwhile for the release this should work as this is going into
> > >>>mcsdk meta-ti repo. But to push this to meta-ti, the above change is
> > >>>needed and hence need your input.
> > >>Murali,
> > >>
> > >>I know Carlos has been working on adding SPL support to u-boot already:
> > >>
> > >>http://arago-project.org/pipermail/meta-arago/2013-February/000937.html
> > >>
> > >>Can you sync up with him and come up with a unified solution for both SPL and
> > >>GPH images that works for everybody and I'd be happy to push it to meta-ti.
> > >>
> > >>Also, I'd like to hear Tom's opinion on this, as current U-Boot maintainer -
> > >>if there is a better approach to this exists or planned in mainline.
> > >I haven't seen the keystone patches yet, so I don't know if the GPH
> > >image is "special" in terms of headers, but the build target
> > >u-boot-with-spl.bin is the canonical build target for "put
> > >u-boot-spl.bin and u-boot.bin together".
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
> 

Re: [PATCH] recipe: u-boot: update recipe for more images

[Cyril Chemparathy]

On 02/26/2013 02:53 PM, Murali Karicheri wrote:
> + Cyril,
>
> Cyril had worked on adding GPH format for u-boot. Want to hear from him
> why not used the u-boot-with-spl.bin build target.
>

Because u-boot-with-spl.bin does not wrap the spl image with a header 
that the rom bootloader understands.

> Murali
> On 2/26/2013 2:31 PM, Tom Rini wrote:
>> On 02/26/2013 02:20 PM, Denys Dmytriyenko wrote:
>>> On Tue, Feb 26, 2013 at 01:42:31PM -0500, Murali Karicheri wrote:
>>>> Denys,
>>>>
>>>> I understand that this recipe is  not complete as u-boot_2013.01.bb
>>>> may be shared by other machines and this change is not applicable.
>>>> Do I need to add this to some other file? Please advise.
>>>>
>>>> Meanwhile for the release this should work as this is going into
>>>> mcsdk meta-ti repo. But to push this to meta-ti, the above change is
>>>> needed and hence need your input.
>>> Murali,
>>>
>>> I know Carlos has been working on adding SPL support to u-boot already:
>>>
>>> http://arago-project.org/pipermail/meta-arago/2013-February/000937.html
>>>
>>> Can you sync up with him and come up with a unified solution for both
>>> SPL and
>>> GPH images that works for everybody and I'd be happy to push it to
>>> meta-ti.
>>>
>>> Also, I'd like to hear Tom's opinion on this, as current U-Boot
>>> maintainer -
>>> if there is a better approach to this exists or planned in mainline.
>> I haven't seen the keystone patches yet, so I don't know if the GPH
>> image is "special" in terms of headers, but the build target
>> u-boot-with-spl.bin is the canonical build target for "put
>> u-boot-spl.bin and u-boot.bin together".
>>
>

Re: [PATCH] recipe: u-boot: update recipe for more images

[Tom Rini]

On 02/26/2013 03:12 PM, Cyril Chemparathy wrote:
> On 02/26/2013 02:53 PM, Murali Karicheri wrote:
>> + Cyril,
>>
>> Cyril had worked on adding GPH format for u-boot. Want to hear from him
>> why not used the u-boot-with-spl.bin build target.
>>
> 
> Because u-boot-with-spl.bin does not wrap the spl image with a header
> that the rom bootloader understands.

What is the header we need?  I assume it's not compatible with our
previous headers.  And if it's just a convenience for programming the
flash, maybe we just stick with two files, to avoid further headaches
upstream?  It's a general rule that concatenating files would make
programming easier, but it's a small savings on either end, yes?  This
would also make the packaging easier since it would just be
SPL_IMAGE_FILE (or whatever) set to u-boot-spl.gph and so forth.  Thanks!

-- 
Tom

Re: [PATCH] recipe: u-boot: update recipe for more images

[Carlos Hernandez]

On 02/26/2013 03:26 PM, Tom Rini wrote:
> On 02/26/2013 03:12 PM, Cyril Chemparathy wrote:
>> On 02/26/2013 02:53 PM, Murali Karicheri wrote:
>>> + Cyril,
>>>
>>> Cyril had worked on adding GPH format for u-boot. Want to hear from him
>>> why not used the u-boot-with-spl.bin build target.
>>>
>> Because u-boot-with-spl.bin does not wrap the spl image with a header
>> that the rom bootloader understands.
> What is the header we need?  I assume it's not compatible with our
> previous headers.  And if it's just a convenience for programming the
> flash, maybe we just stick with two files, to avoid further headaches
> upstream?  It's a general rule that concatenating files would make
> programming easier, but it's a small savings on either end, yes?  This
> would also make the packaging easier since it would just be
> SPL_IMAGE_FILE (or whatever) set to u-boot-spl.gph and so forth.  Thanks!
>
I believe this is what the patch I submitted (to meta-arago) few days 
ago do.
You need to define SPL_UART_BINARY in your recipe to enable the extra 
do_install and do_deploy functionality.

Re: [PATCH] recipe: u-boot: update recipe for more images

[Denys Dmytriyenko]

On Tue, Feb 26, 2013 at 03:38:03PM -0500, Carlos Hernandez wrote:
> On 02/26/2013 03:26 PM, Tom Rini wrote:
> >On 02/26/2013 03:12 PM, Cyril Chemparathy wrote:
> >>On 02/26/2013 02:53 PM, Murali Karicheri wrote:
> >>>+ Cyril,
> >>>
> >>>Cyril had worked on adding GPH format for u-boot. Want to hear from him
> >>>why not used the u-boot-with-spl.bin build target.
> >>>
> >>Because u-boot-with-spl.bin does not wrap the spl image with a header
> >>that the rom bootloader understands.
> >What is the header we need?  I assume it's not compatible with our
> >previous headers.  And if it's just a convenience for programming the
> >flash, maybe we just stick with two files, to avoid further headaches
> >upstream?  It's a general rule that concatenating files would make
> >programming easier, but it's a small savings on either end, yes?  This
> >would also make the packaging easier since it would just be
> >SPL_IMAGE_FILE (or whatever) set to u-boot-spl.gph and so forth.  Thanks!
> >
> I believe this is what the patch I submitted (to meta-arago) few
> days ago do.
> You need to define SPL_UART_BINARY in your recipe to enable the
> extra do_install and do_deploy functionality.

Correct, but it doesn't support GPH yet, only SPL, AFAIK. Can you look into 
extending it in such a way it can be used for any/all of the use cases easily?

-- 
Denys

Re: [PATCH] recipe: u-boot: update recipe for more images

[Tom Rini]

On 02/26/2013 04:22 PM, Denys Dmytriyenko wrote:
> On Tue, Feb 26, 2013 at 03:38:03PM -0500, Carlos Hernandez wrote:
>> On 02/26/2013 03:26 PM, Tom Rini wrote:
>>> On 02/26/2013 03:12 PM, Cyril Chemparathy wrote:
>>>> On 02/26/2013 02:53 PM, Murali Karicheri wrote:
>>>>> + Cyril,
>>>>>
>>>>> Cyril had worked on adding GPH format for u-boot. Want to hear from him
>>>>> why not used the u-boot-with-spl.bin build target.
>>>>>
>>>> Because u-boot-with-spl.bin does not wrap the spl image with a header
>>>> that the rom bootloader understands.
>>> What is the header we need?  I assume it's not compatible with our
>>> previous headers.  And if it's just a convenience for programming the
>>> flash, maybe we just stick with two files, to avoid further headaches
>>> upstream?  It's a general rule that concatenating files would make
>>> programming easier, but it's a small savings on either end, yes?  This
>>> would also make the packaging easier since it would just be
>>> SPL_IMAGE_FILE (or whatever) set to u-boot-spl.gph and so forth.  Thanks!
>>>
>> I believe this is what the patch I submitted (to meta-arago) few
>> days ago do.
>> You need to define SPL_UART_BINARY in your recipe to enable the
>> extra do_install and do_deploy functionality.
> 
> Correct, but it doesn't support GPH yet, only SPL, AFAIK. Can you look into 
> extending it in such a way it can be used for any/all of the use cases easily?

Well, if it's just a header on the SPL binary we already have that as "MLO".

-- 
Tom

Re: [PATCH] recipe: u-boot: update recipe for more images

[Hernandez, Carlos]

> I haven't seen the keystone patches yet, so I don't know if the GPH
> image is "special" in terms of headers, but the build target
> u-boot-with-spl.bin is the canonical build target for "put
> u-boot-spl.bin and u-boot.bin together".
> 

The current test script transfers (via UART) u-boot-spl.bin and then u-boot.img. 
Have anybody tried booting one omap or sitara device using 'u-boot-with-spl.bin' image.

> --
> Tom

Re: [PATCH] recipe: u-boot: update recipe for more images

[Tom Rini]

On 02/26/2013 03:13 PM, Hernandez, Carlos wrote:
>> I haven't seen the keystone patches yet, so I don't know if the GPH
>> image is "special" in terms of headers, but the build target
>> u-boot-with-spl.bin is the canonical build target for "put
>> u-boot-spl.bin and u-boot.bin together".
>>
> 
> The current test script transfers (via UART) u-boot-spl.bin and then u-boot.img. 
> Have anybody tried booting one omap or sitara device using 'u-boot-with-spl.bin' image.

No, because none of the boot modes that take u-boot-spl.bin would like
to see a concatenated image like that.  That build target is primarily
used on other (vendors) platforms.

-- 
Tom