All of lore.kernel.org
 help / color / mirror / Atom feed
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!



      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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.