* [PATCH] u-boot.inc: allow variable make targets
@ 2011-03-15 14:42 Chase Maupin
2011-03-15 15:16 ` Maupin, Chase
0 siblings, 1 reply; 5+ messages in thread
From: Chase Maupin @ 2011-03-15 14:42 UTC (permalink / raw)
To: openembedded-devel; +Cc: Chase Maupin
* Later versions of u-boot sometimes have top level make targets
such as u-boot.imx, u-boot.kwb, and for upcoming TI devices
u-boot.ti.
* These targets define different calls to the mkimage tool to
change the images created, such as changing the load address.
* This change allows recipes to change the default make target
of "all" to something like u-boot.ti or u-boot.imx by setting
the UBOOT_MAKE_TARGET variable.
Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
---
recipes/u-boot/u-boot.inc | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/recipes/u-boot/u-boot.inc b/recipes/u-boot/u-boot.inc
index 88c6a4d..ffd1e1e 100644
--- a/recipes/u-boot/u-boot.inc
+++ b/recipes/u-boot/u-boot.inc
@@ -16,6 +16,7 @@ UBOOT_MACHINE ?= "${MACHINE}_config"
UBOOT_BINARY ?= "u-boot.bin"
UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.bin"
UBOOT_SYMLINK ?= "u-boot-${MACHINE}.bin"
+UBOOT_MAKE_TARGET ?= "all"
do_configure () {
oe_runmake ${UBOOT_MACHINE}
@@ -25,7 +26,7 @@ do_compile () {
unset LDFLAGS
unset CFLAGS
unset CPPFLAGS
- oe_runmake all
+ oe_runmake ${UBOOT_MAKE_TARGET}
oe_runmake tools env HOSTCC="${CC}"
}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] u-boot.inc: allow variable make targets
2011-03-15 14:42 [PATCH] u-boot.inc: allow variable make targets Chase Maupin
@ 2011-03-15 15:16 ` Maupin, Chase
2011-03-16 13:40 ` Maupin, Chase
0 siblings, 1 reply; 5+ messages in thread
From: Maupin, Chase @ 2011-03-15 15:16 UTC (permalink / raw)
To: Chase Maupin, openembedded-devel
I should have probably clarified that this was intended for the 2011.03-maintenance branch. Tom, I have also submitted this to the oe-core. Would you be willing to pick this up?
Sincerely,
Chase Maupin
Open Platforms
ARM MPU
e-mail: chase.maupin@ti.com
phone: (214) 567-2950
For support:
Forums - http://community.ti.com/forums/
Wiki - http://wiki.davincidsp.com/
> -----Original Message-----
> From: Chase Maupin [mailto:chasemaupin03@gmail.com]
> Sent: Tuesday, March 15, 2011 9:42 AM
> To: openembedded-devel@lists.openembedded.org
> Cc: Maupin, Chase
> Subject: [PATCH] u-boot.inc: allow variable make targets
>
> * Later versions of u-boot sometimes have top level make targets
> such as u-boot.imx, u-boot.kwb, and for upcoming TI devices
> u-boot.ti.
> * These targets define different calls to the mkimage tool to
> change the images created, such as changing the load address.
> * This change allows recipes to change the default make target
> of "all" to something like u-boot.ti or u-boot.imx by setting
> the UBOOT_MAKE_TARGET variable.
>
> Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
> ---
> recipes/u-boot/u-boot.inc | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/recipes/u-boot/u-boot.inc b/recipes/u-boot/u-boot.inc
> index 88c6a4d..ffd1e1e 100644
> --- a/recipes/u-boot/u-boot.inc
> +++ b/recipes/u-boot/u-boot.inc
> @@ -16,6 +16,7 @@ UBOOT_MACHINE ?= "${MACHINE}_config"
> UBOOT_BINARY ?= "u-boot.bin"
> UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.bin"
> UBOOT_SYMLINK ?= "u-boot-${MACHINE}.bin"
> +UBOOT_MAKE_TARGET ?= "all"
>
> do_configure () {
> oe_runmake ${UBOOT_MACHINE}
> @@ -25,7 +26,7 @@ do_compile () {
> unset LDFLAGS
> unset CFLAGS
> unset CPPFLAGS
> - oe_runmake all
> + oe_runmake ${UBOOT_MAKE_TARGET}
> oe_runmake tools env HOSTCC="${CC}"
> }
>
> --
> 1.7.0.4
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] u-boot.inc: allow variable make targets
2011-03-15 15:16 ` Maupin, Chase
@ 2011-03-16 13:40 ` Maupin, Chase
0 siblings, 0 replies; 5+ messages in thread
From: Maupin, Chase @ 2011-03-16 13:40 UTC (permalink / raw)
To: openembedded-devel, Chase Maupin
> -----Original Message-----
> From: openembedded-devel-bounces@lists.openembedded.org
> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
> Maupin, Chase
> Sent: Tuesday, March 15, 2011 10:16 AM
> To: Chase Maupin; openembedded-devel@lists.openembedded.org
> Subject: Re: [oe] [PATCH] u-boot.inc: allow variable make targets
>
> I should have probably clarified that this was intended for the 2011.03-
> maintenance branch. Tom, I have also submitted this to the oe-core.
> Would you be willing to pick this up?
Just FYI this has been merged into the oe-core master (see http://lists.linuxtogo.org/pipermail/openembedded-core/2011-March/000533.html) as well as the angstrom-layers master (see http://lists.linuxtogo.org/pipermail/openembedded-core/2011-March/000512.html)
>
> > -----Original Message-----
> > From: Chase Maupin [mailto:chasemaupin03@gmail.com]
> > Sent: Tuesday, March 15, 2011 9:42 AM
> > To: openembedded-devel@lists.openembedded.org
> > Cc: Maupin, Chase
> > Subject: [PATCH] u-boot.inc: allow variable make targets
> >
> > * Later versions of u-boot sometimes have top level make targets
> > such as u-boot.imx, u-boot.kwb, and for upcoming TI devices
> > u-boot.ti.
> > * These targets define different calls to the mkimage tool to
> > change the images created, such as changing the load address.
> > * This change allows recipes to change the default make target
> > of "all" to something like u-boot.ti or u-boot.imx by setting
> > the UBOOT_MAKE_TARGET variable.
> >
> > Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
> > ---
> > recipes/u-boot/u-boot.inc | 3 ++-
> > 1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/recipes/u-boot/u-boot.inc b/recipes/u-boot/u-boot.inc
> > index 88c6a4d..ffd1e1e 100644
> > --- a/recipes/u-boot/u-boot.inc
> > +++ b/recipes/u-boot/u-boot.inc
> > @@ -16,6 +16,7 @@ UBOOT_MACHINE ?= "${MACHINE}_config"
> > UBOOT_BINARY ?= "u-boot.bin"
> > UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.bin"
> > UBOOT_SYMLINK ?= "u-boot-${MACHINE}.bin"
> > +UBOOT_MAKE_TARGET ?= "all"
> >
> > do_configure () {
> > oe_runmake ${UBOOT_MACHINE}
> > @@ -25,7 +26,7 @@ do_compile () {
> > unset LDFLAGS
> > unset CFLAGS
> > unset CPPFLAGS
> > - oe_runmake all
> > + oe_runmake ${UBOOT_MAKE_TARGET}
> > oe_runmake tools env HOSTCC="${CC}"
> > }
> >
> > --
> > 1.7.0.4
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] u-boot.inc: allow variable make targets
2011-03-15 13:31 Chase Maupin
@ 2011-03-16 13:18 ` Richard Purdie
0 siblings, 0 replies; 5+ messages in thread
From: Richard Purdie @ 2011-03-16 13:18 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Chase Maupin
On Tue, 2011-03-15 at 08:31 -0500, Chase Maupin wrote:
> * Later versions of u-boot sometimes have top level make targets
> such as u-boot.imx, u-boot.kwb, and for upcoming TI devices
> u-boot.ti.
> * These targets define different calls to the mkimage tool to
> change the images created, such as changing the load address.
> * This change allows recipes to change the default make target
> of "all" to something like u-boot.ti or u-boot.imx by setting
> the UBOOT_MAKE_TARGET variable.
>
> Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
> ---
> meta/recipes-bsp/uboot/u-boot.inc | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
Merged to master, thanks.
Richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] u-boot.inc: allow variable make targets
@ 2011-03-15 13:31 Chase Maupin
2011-03-16 13:18 ` Richard Purdie
0 siblings, 1 reply; 5+ messages in thread
From: Chase Maupin @ 2011-03-15 13:31 UTC (permalink / raw)
To: openembedded-core; +Cc: Chase Maupin
* Later versions of u-boot sometimes have top level make targets
such as u-boot.imx, u-boot.kwb, and for upcoming TI devices
u-boot.ti.
* These targets define different calls to the mkimage tool to
change the images created, such as changing the load address.
* This change allows recipes to change the default make target
of "all" to something like u-boot.ti or u-boot.imx by setting
the UBOOT_MAKE_TARGET variable.
Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
---
meta/recipes-bsp/uboot/u-boot.inc | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/meta/recipes-bsp/uboot/u-boot.inc b/meta/recipes-bsp/uboot/u-boot.inc
index d1cebee..058e3ba 100644
--- a/meta/recipes-bsp/uboot/u-boot.inc
+++ b/meta/recipes-bsp/uboot/u-boot.inc
@@ -14,13 +14,14 @@ EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} OPTFLAGS='-O2'"
UBOOT_MACHINE ?= "${MACHINE}_config"
UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.bin"
UBOOT_SYMLINK ?= "u-boot-${MACHINE}.bin"
+UBOOT_MAKE_TARGET ?= "all"
do_compile () {
unset LDFLAGS
unset CFLAGS
unset CPPFLAGS
oe_runmake ${UBOOT_MACHINE}
- oe_runmake all
+ oe_runmake ${UBOOT_MAKE_TARGET}
}
do_deploy () {
--
1.7.0.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-03-16 13:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-15 14:42 [PATCH] u-boot.inc: allow variable make targets Chase Maupin
2011-03-15 15:16 ` Maupin, Chase
2011-03-16 13:40 ` Maupin, Chase
-- strict thread matches above, loose matches on Subject: below --
2011-03-15 13:31 Chase Maupin
2011-03-16 13:18 ` Richard Purdie
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.