From: Saul Wold <sgw@linux.intel.com>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Cc: Chase Maupin <Chase.Maupin@ti.com>
Subject: Re: [PATCH] u-boot inc: update inc file for newer u-boot versions
Date: Tue, 07 Feb 2012 20:05:56 -0800 [thread overview]
Message-ID: <4F31F4A4.7010002@linux.intel.com> (raw)
In-Reply-To: <1328537818-23228-1-git-send-email-Chase.Maupin@ti.com>
On 02/06/2012 06:16 AM, Chase Maupin wrote:
> * Add support for different suffixes found with later u-boot
> versions which have switched from .bin to .img
> * Allow recipes that include u-boot.inc to also package an
> SPL if they build one.
> * Minimum requirement is to set the SPL_BINARY value to add
> the SPL to the u-boot package as well as into the deploy
> directory.
>
> Signed-off-by: Chase Maupin<Chase.Maupin@ti.com>
> ---
> meta/recipes-bsp/u-boot/u-boot.inc | 35 +++++++++++++++++++++++++++++++----
> 1 files changed, 31 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
> index 4180345..700d5d3 100644
> --- a/meta/recipes-bsp/u-boot/u-boot.inc
> +++ b/meta/recipes-bsp/u-boot/u-boot.inc
> @@ -19,11 +19,22 @@ python () {
> raise bb.parse.SkipPackage("because UBOOT_MACHINE is not set")
> }
>
> -UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.bin"
> -UBOOT_BINARY ?= "u-boot.bin"
> -UBOOT_SYMLINK ?= "u-boot-${MACHINE}.bin"
> +# Some versions of u-boot use .bin and others use .img. By default use .bin
> +# but enable individual recipes to change this value.
> +UBOOT_SUFFIX ?= "bin"
> +UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.${UBOOT_SUFFIX}"
> +UBOOT_BINARY ?= "u-boot.${UBOOT_SUFFIX}"
> +UBOOT_SYMLINK ?= "u-boot-${MACHINE}.${UBOOT_SUFFIX}"
> UBOOT_MAKE_TARGET ?= "all"
>
> +# Some versions of u-boot build an SPL (Second Program Loader) image that
> +# should be packaged along with the u-boot binary as well as placed in the
> +# deploy directory. For those versions they can set the following variables
> +# to allow packaging the SPL.
> +SPL_BINARY ?= ""
> +SPL_IMAGE ?= "${SPL_BINARY}-${PV}-${PR}"
> +SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}"
> +
> do_compile () {
> unset LDFLAGS
> unset CFLAGS
> @@ -36,15 +47,31 @@ do_install () {
> install -d ${D}/boot
> install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
> ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
> +
> + if [ "x${SPL_BINARY}" != "x" ]
> + then
> + install ${S}/${SPL_BINARY} ${D}/boot/${SPL_IMAGE}
> + ln -sf ${SPL_IMAGE} ${D}/boot/${SPL_BINARY}
> + fi
> }
>
> FILES_${PN} = "/boot"
>
> do_deploy () {
> - install ${S}/u-boot.bin ${DEPLOYDIR}/${UBOOT_IMAGE}
> + install -d ${DEPLOYDIR}
> + install ${S}/${UBOOT_BINARY} ${DEPLOYDIR}/${UBOOT_IMAGE}
>
> cd ${DEPLOYDIR}
> rm -f ${UBOOT_SYMLINK}
> ln -sf ${UBOOT_IMAGE} ${UBOOT_SYMLINK}
> +
> + if [ "x${SPL_BINARY}" != "x" ]
> + then
> + install ${S}/${SPL_BINARY} ${DEPLOYDIR}/${SPL_IMAGE}
> + rm -f ${DEPLOYDIR}/${SPL_BINARY} ${DEPLOYDIR}/${SPL_SYMLINK}
> + ln -sf ${SPL_IMAGE} ${DEPLOYDIR}/${SPL_BINARY}
> + ln -sf ${SPL_IMAGE} ${DEPLOYDIR}/${SPL_SYMLINK}
> + fi
> }
> +
> addtask deploy before do_build after do_compile
Merged into OE-core
Thanks
Sau!
prev parent reply other threads:[~2012-02-08 4:14 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-06 14:16 [PATCH] u-boot inc: update inc file for newer u-boot versions Chase Maupin
2012-02-08 4:05 ` Saul Wold [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F31F4A4.7010002@linux.intel.com \
--to=sgw@linux.intel.com \
--cc=Chase.Maupin@ti.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox