* [PATCH] u-boot: add machine name to spl image name @ 2012-04-05 11:48 Stefan Herbrechtsmeier 2012-04-06 3:18 ` Saul Wold 0 siblings, 1 reply; 10+ messages in thread From: Stefan Herbrechtsmeier @ 2012-04-05 11:48 UTC (permalink / raw) To: openembedded-core Signed-off-by: Stefan Herbrechtsmeier <stefan@herbrechtsmeier.net> --- meta/recipes-bsp/u-boot/u-boot.inc | 2 +- meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc index 700d5d3..0445c34 100644 --- a/meta/recipes-bsp/u-boot/u-boot.inc +++ b/meta/recipes-bsp/u-boot/u-boot.inc @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" do_compile () { diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb index 1ebdbea..e99bc2c 100644 --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" PV = "v2011.03+git${SRCPV}" -PR = "r5" +PR = "r6" SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb index 8ebdbff..680401f 100644 --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" PV = "v2011.06+git${SRCPV}" -PR = "r0" +PR = "r1" SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-05 11:48 [PATCH] u-boot: add machine name to spl image name Stefan Herbrechtsmeier @ 2012-04-06 3:18 ` Saul Wold 2012-04-06 13:29 ` Koen Kooi 0 siblings, 1 reply; 10+ messages in thread From: Saul Wold @ 2012-04-06 3:18 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: > Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> > --- > meta/recipes-bsp/u-boot/u-boot.inc | 2 +- > meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- > meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc > index 700d5d3..0445c34 100644 > --- a/meta/recipes-bsp/u-boot/u-boot.inc > +++ b/meta/recipes-bsp/u-boot/u-boot.inc > @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" > # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" > SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" > > do_compile () { > diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > index 1ebdbea..e99bc2c 100644 > --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" > > PV = "v2011.03+git${SRCPV}" > -PR = "r5" > +PR = "r6" > > SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > > diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > index 8ebdbff..680401f 100644 > --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" > > PV = "v2011.06+git${SRCPV}" > -PR = "r0" > +PR = "r1" > > SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > Merged into OE-Core Thanks Sau! ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-06 3:18 ` Saul Wold @ 2012-04-06 13:29 ` Koen Kooi 2012-04-10 13:41 ` Richard Purdie 0 siblings, 1 reply; 10+ messages in thread From: Koen Kooi @ 2012-04-06 13:29 UTC (permalink / raw) To: Patches and discussions about the oe-core layer Cc: Patches and discussions about the oe-core layer Op 5 apr. 2012 om 20:18 heeft Saul Wold <sgw@linux.intel.com> het volgende geschreven: > On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: >> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> >> --- >> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- >> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- >> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- >> 3 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc >> index 700d5d3..0445c34 100644 >> --- a/meta/recipes-bsp/u-boot/u-boot.inc >> +++ b/meta/recipes-bsp/u-boot/u-boot.inc >> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" >> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" >> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" >> >> do_compile () { >> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >> index 1ebdbea..e99bc2c 100644 >> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" >> >> PV = "v2011.03+git${SRCPV}" >> -PR = "r5" >> +PR = "r6" >> >> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >> >> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >> index 8ebdbff..680401f 100644 >> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" >> >> PV = "v2011.06+git${SRCPV}" >> -PR = "r0" >> +PR = "r1" >> >> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >> > > Merged into OE-Core wait a minute, why was this merged? > > Thanks > Sau! > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-06 13:29 ` Koen Kooi @ 2012-04-10 13:41 ` Richard Purdie 2012-04-10 13:56 ` Koen Kooi 0 siblings, 1 reply; 10+ messages in thread From: Richard Purdie @ 2012-04-10 13:41 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: > > Op 5 apr. 2012 om 20:18 heeft Saul Wold <sgw@linux.intel.com> het volgende geschreven: > > > On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: > >> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> > >> --- > >> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- > >> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- > >> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- > >> 3 files changed, 3 insertions(+), 3 deletions(-) > >> > >> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc > >> index 700d5d3..0445c34 100644 > >> --- a/meta/recipes-bsp/u-boot/u-boot.inc > >> +++ b/meta/recipes-bsp/u-boot/u-boot.inc > >> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" > >> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" > >> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" > >> > >> do_compile () { > >> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >> index 1ebdbea..e99bc2c 100644 > >> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > >> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" > >> > >> PV = "v2011.03+git${SRCPV}" > >> -PR = "r5" > >> +PR = "r6" > >> > >> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > >> > >> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >> index 8ebdbff..680401f 100644 > >> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > >> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" > >> > >> PV = "v2011.06+git${SRCPV}" > >> -PR = "r0" > >> +PR = "r1" > >> > >> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > >> > > > > Merged into OE-Core > > wait a minute, why was this merged? In retrospect it shouldn't. It was presented to me as part of a series and I wasn't careful enough about sifting through the commits, likely the jetlag wasn't helping. I'll aim to try harder, people can help by pointing out patches they don't think should be going into the release at this point... Cheers, Richard ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-10 13:41 ` Richard Purdie @ 2012-04-10 13:56 ` Koen Kooi 2012-04-10 14:19 ` Stefan Herbrechtsmeier 2012-04-11 11:36 ` Richard Purdie 0 siblings, 2 replies; 10+ messages in thread From: Koen Kooi @ 2012-04-10 13:56 UTC (permalink / raw) To: Patches and discussions about the oe-core layer Op 10 apr. 2012, om 15:41 heeft Richard Purdie het volgende geschreven: > On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: >> >> Op 5 apr. 2012 om 20:18 heeft Saul Wold <sgw@linux.intel.com> het volgende geschreven: >> >>> On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: >>>> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> >>>> --- >>>> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- >>>> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- >>>> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- >>>> 3 files changed, 3 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc >>>> index 700d5d3..0445c34 100644 >>>> --- a/meta/recipes-bsp/u-boot/u-boot.inc >>>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc >>>> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" >>>> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" >>>> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" >>>> >>>> do_compile () { >>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>> index 1ebdbea..e99bc2c 100644 >>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" >>>> >>>> PV = "v2011.03+git${SRCPV}" >>>> -PR = "r5" >>>> +PR = "r6" >>>> >>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>> >>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>> index 8ebdbff..680401f 100644 >>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" >>>> >>>> PV = "v2011.06+git${SRCPV}" >>>> -PR = "r0" >>>> +PR = "r1" >>>> >>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>> >>> >>> Merged into OE-Core >> >> wait a minute, why was this merged? > > In retrospect it shouldn't. Can we back it out? It breaks a ton of buildscripts. SPL_SYMLINK already has the machine. > It was presented to me as part of a series > and I wasn't careful enough about sifting through the commits, likely > the jetlag wasn't helping. I'll aim to try harder, people can help by > pointing out patches they don't think should be going into the release > at this point... I read Sauls mail at the airport at 6:30 in the morning, which means that if I skipped sleeping I would have had a 16 hour window to object. For a bootloader patch that's not enough. Especially when the people who care about bootloaders are on spring break or attending collab. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-10 13:56 ` Koen Kooi @ 2012-04-10 14:19 ` Stefan Herbrechtsmeier 2012-04-10 14:22 ` Koen Kooi 2012-04-11 11:36 ` Richard Purdie 1 sibling, 1 reply; 10+ messages in thread From: Stefan Herbrechtsmeier @ 2012-04-10 14:19 UTC (permalink / raw) To: Patches and discussions about the oe-core layer Am 10.04.2012 15:56, schrieb Koen Kooi: > Op 10 apr. 2012, om 15:41 heeft Richard Purdie het volgende geschreven: > >> On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: >>> Op 5 apr. 2012 om 20:18 heeft Saul Wold<sgw@linux.intel.com> het volgende geschreven: >>> >>>> On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: >>>>> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> >>>>> --- >>>>> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- >>>>> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- >>>>> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- >>>>> 3 files changed, 3 insertions(+), 3 deletions(-) >>>>> >>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc >>>>> index 700d5d3..0445c34 100644 >>>>> --- a/meta/recipes-bsp/u-boot/u-boot.inc >>>>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc >>>>> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" >>>>> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" >>>>> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" >>>>> >>>>> do_compile () { >>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>> index 1ebdbea..e99bc2c 100644 >>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>>> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" >>>>> >>>>> PV = "v2011.03+git${SRCPV}" >>>>> -PR = "r5" >>>>> +PR = "r6" >>>>> >>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>>> >>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>> index 8ebdbff..680401f 100644 >>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>>> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" >>>>> >>>>> PV = "v2011.06+git${SRCPV}" >>>>> -PR = "r0" >>>>> +PR = "r1" >>>>> >>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>>> >>>> Merged into OE-Core >>> wait a minute, why was this merged? >> In retrospect it shouldn't. > Can we back it out? It breaks a ton of buildscripts. SPL_SYMLINK already has the machine. But this means that your SPL_SYMLINK can point to a wrong SPL_IMAGE if you don't use machine in the tmpdir path and build more than one machine. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-10 14:19 ` Stefan Herbrechtsmeier @ 2012-04-10 14:22 ` Koen Kooi 2012-04-10 14:28 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Koen Kooi @ 2012-04-10 14:22 UTC (permalink / raw) To: Patches and discussions about the oe-core layer Op 10 apr. 2012, om 16:19 heeft Stefan Herbrechtsmeier het volgende geschreven: > Am 10.04.2012 15:56, schrieb Koen Kooi: >> Op 10 apr. 2012, om 15:41 heeft Richard Purdie het volgende geschreven: >> >>> On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: >>>> Op 5 apr. 2012 om 20:18 heeft Saul Wold<sgw@linux.intel.com> het volgende geschreven: >>>> >>>>> On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: >>>>>> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> >>>>>> --- >>>>>> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- >>>>>> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- >>>>>> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- >>>>>> 3 files changed, 3 insertions(+), 3 deletions(-) >>>>>> >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc >>>>>> index 700d5d3..0445c34 100644 >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot.inc >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc >>>>>> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" >>>>>> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" >>>>>> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" >>>>>> >>>>>> do_compile () { >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>>> index 1ebdbea..e99bc2c 100644 >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>>>> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" >>>>>> >>>>>> PV = "v2011.03+git${SRCPV}" >>>>>> -PR = "r5" >>>>>> +PR = "r6" >>>>>> >>>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>>>> >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>>> index 8ebdbff..680401f 100644 >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>>>> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" >>>>>> >>>>>> PV = "v2011.06+git${SRCPV}" >>>>>> -PR = "r0" >>>>>> +PR = "r1" >>>>>> >>>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>>>> >>>>> Merged into OE-Core >>>> wait a minute, why was this merged? >>> In retrospect it shouldn't. >> Can we back it out? It breaks a ton of buildscripts. SPL_SYMLINK already has the machine. > But this means that your SPL_SYMLINK can point to a wrong SPL_IMAGE if you don't use machine in the tmpdir path and build more than one machine. I guess I'll have a suggestion for the 1.3 release: [koen@Angstrom-F16-vm-rpm meta-angstrom]$ git grep DEPLOY_DIR_IMAGE conf/distro/include/angstrom.inc:DEPLOY_DIR_IMAGE = "${DEPLOY_DIR}/images/${MACHINE}" ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-10 14:22 ` Koen Kooi @ 2012-04-10 14:28 ` Martin Jansa 0 siblings, 0 replies; 10+ messages in thread From: Martin Jansa @ 2012-04-10 14:28 UTC (permalink / raw) To: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 3486 bytes --] On Tue, Apr 10, 2012 at 04:22:43PM +0200, Koen Kooi wrote: > > Op 10 apr. 2012, om 16:19 heeft Stefan Herbrechtsmeier het volgende geschreven: > > > Am 10.04.2012 15:56, schrieb Koen Kooi: > >> Op 10 apr. 2012, om 15:41 heeft Richard Purdie het volgende geschreven: > >> > >>> On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: > >>>> Op 5 apr. 2012 om 20:18 heeft Saul Wold<sgw@linux.intel.com> het volgende geschreven: > >>>> > >>>>> On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: > >>>>>> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> > >>>>>> --- > >>>>>> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- > >>>>>> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- > >>>>>> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- > >>>>>> 3 files changed, 3 insertions(+), 3 deletions(-) > >>>>>> > >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc > >>>>>> index 700d5d3..0445c34 100644 > >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot.inc > >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc > >>>>>> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" > >>>>>> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" > >>>>>> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" > >>>>>> > >>>>>> do_compile () { > >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >>>>>> index 1ebdbea..e99bc2c 100644 > >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >>>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > >>>>>> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" > >>>>>> > >>>>>> PV = "v2011.03+git${SRCPV}" > >>>>>> -PR = "r5" > >>>>>> +PR = "r6" > >>>>>> > >>>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > >>>>>> > >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >>>>>> index 8ebdbff..680401f 100644 > >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >>>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > >>>>>> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" > >>>>>> > >>>>>> PV = "v2011.06+git${SRCPV}" > >>>>>> -PR = "r0" > >>>>>> +PR = "r1" > >>>>>> > >>>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > >>>>>> > >>>>> Merged into OE-Core > >>>> wait a minute, why was this merged? > >>> In retrospect it shouldn't. > >> Can we back it out? It breaks a ton of buildscripts. SPL_SYMLINK already has the machine. > > But this means that your SPL_SYMLINK can point to a wrong SPL_IMAGE if you don't use machine in the tmpdir path and build more than one machine. > > I guess I'll have a suggestion for the 1.3 release: > > [koen@Angstrom-F16-vm-rpm meta-angstrom]$ git grep DEPLOY_DIR_IMAGE > conf/distro/include/angstrom.inc:DEPLOY_DIR_IMAGE = "${DEPLOY_DIR}/images/${MACHINE}" Looks good :) meta-shr/conf/distro/shr.conf:DEPLOY_DIR_IMAGE = "${DEPLOY_DIR}/images/${MACHINE}" -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 205 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-10 13:56 ` Koen Kooi 2012-04-10 14:19 ` Stefan Herbrechtsmeier @ 2012-04-11 11:36 ` Richard Purdie 2012-04-11 11:49 ` Koen Kooi 1 sibling, 1 reply; 10+ messages in thread From: Richard Purdie @ 2012-04-11 11:36 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On Tue, 2012-04-10 at 15:56 +0200, Koen Kooi wrote: > Op 10 apr. 2012, om 15:41 heeft Richard Purdie het volgende geschreven: > > > On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: > >> > >> Op 5 apr. 2012 om 20:18 heeft Saul Wold <sgw@linux.intel.com> het volgende geschreven: > >> > >>> On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: > >>>> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> > >>>> --- > >>>> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- > >>>> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- > >>>> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- > >>>> 3 files changed, 3 insertions(+), 3 deletions(-) > >>>> > >>>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc > >>>> index 700d5d3..0445c34 100644 > >>>> --- a/meta/recipes-bsp/u-boot/u-boot.inc > >>>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc > >>>> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" > >>>> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" > >>>> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" > >>>> > >>>> do_compile () { > >>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >>>> index 1ebdbea..e99bc2c 100644 > >>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb > >>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > >>>> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" > >>>> > >>>> PV = "v2011.03+git${SRCPV}" > >>>> -PR = "r5" > >>>> +PR = "r6" > >>>> > >>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > >>>> > >>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >>>> index 8ebdbff..680401f 100644 > >>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb > >>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" > >>>> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" > >>>> > >>>> PV = "v2011.06+git${SRCPV}" > >>>> -PR = "r0" > >>>> +PR = "r1" > >>>> > >>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" > >>>> > >>> > >>> Merged into OE-Core > >> > >> wait a minute, why was this merged? > > > > In retrospect it shouldn't. > > Can we back it out? It breaks a ton of buildscripts. SPL_SYMLINK already has the machine. > > > It was presented to me as part of a series > > and I wasn't careful enough about sifting through the commits, likely > > the jetlag wasn't helping. I'll aim to try harder, people can help by > > pointing out patches they don't think should be going into the release > > at this point... > > I read Sauls mail at the airport at 6:30 in the morning, which means > that if I skipped sleeping I would have had a 16 hour window to > object. For a bootloader patch that's not enough. Especially when the > people who care about bootloaders are on spring break or attending > collab. I don't expect people to wait for Saul's emails and then review patches already on the mailing list. Regardless, I'm rather torn in this case. Stefan's usecase is a valid one and I don't see why we have to change the layout of DEPLOY_DIR because of this one issue. I know of people doing some interesting things with layouts and I don't really want to impose policy on that which the change you propose does. Having the symlink contain MACHINE but not the name is rather odd. I also agree its late in the cycle for something which breaks build scripts however :(. So I'm probably in favour of reverting from 1.2 but adding straight away for 1.3 although I don't like letting 1.2 out with this issue present really. Cheers, Richard ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] u-boot: add machine name to spl image name 2012-04-11 11:36 ` Richard Purdie @ 2012-04-11 11:49 ` Koen Kooi 0 siblings, 0 replies; 10+ messages in thread From: Koen Kooi @ 2012-04-11 11:49 UTC (permalink / raw) To: Patches and discussions about the oe-core layer Op 11 apr. 2012, om 13:36 heeft Richard Purdie het volgende geschreven: > On Tue, 2012-04-10 at 15:56 +0200, Koen Kooi wrote: >> Op 10 apr. 2012, om 15:41 heeft Richard Purdie het volgende geschreven: >> >>> On Fri, 2012-04-06 at 06:29 -0700, Koen Kooi wrote: >>>> >>>> Op 5 apr. 2012 om 20:18 heeft Saul Wold <sgw@linux.intel.com> het volgende geschreven: >>>> >>>>> On 04/05/2012 04:48 AM, Stefan Herbrechtsmeier wrote: >>>>>> Signed-off-by: Stefan Herbrechtsmeier<stefan@herbrechtsmeier.net> >>>>>> --- >>>>>> meta/recipes-bsp/u-boot/u-boot.inc | 2 +- >>>>>> meta/recipes-bsp/u-boot/u-boot_2011.03.bb | 2 +- >>>>>> meta/recipes-bsp/u-boot/u-boot_2011.06.bb | 2 +- >>>>>> 3 files changed, 3 insertions(+), 3 deletions(-) >>>>>> >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc >>>>>> index 700d5d3..0445c34 100644 >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot.inc >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc >>>>>> @@ -32,7 +32,7 @@ UBOOT_MAKE_TARGET ?= "all" >>>>>> # 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_IMAGE ?= "${SPL_BINARY}-${MACHINE}-${PV}-${PR}" >>>>>> SPL_SYMLINK ?= "${SPL_BINARY}-${MACHINE}" >>>>>> >>>>>> do_compile () { >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>>> index 1ebdbea..e99bc2c 100644 >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb >>>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>>>> SRCREV = "19b54a701811220221fc4d5089a2bb18892018ca" >>>>>> >>>>>> PV = "v2011.03+git${SRCPV}" >>>>>> -PR = "r5" >>>>>> +PR = "r6" >>>>>> >>>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>>>> >>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>>> index 8ebdbff..680401f 100644 >>>>>> --- a/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2011.06.bb >>>>>> @@ -17,7 +17,7 @@ FILESDIR = "${@os.path.dirname(d.getVar('FILE',1))}/u-boot-git/${MACHINE}" >>>>>> SRCREV = "b1af6f532e0d348b153d5c148369229d24af361a" >>>>>> >>>>>> PV = "v2011.06+git${SRCPV}" >>>>>> -PR = "r0" >>>>>> +PR = "r1" >>>>>> >>>>>> SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git" >>>>>> >>>>> >>>>> Merged into OE-Core >>>> >>>> wait a minute, why was this merged? >>> >>> In retrospect it shouldn't. >> >> Can we back it out? It breaks a ton of buildscripts. SPL_SYMLINK already has the machine. >> >>> It was presented to me as part of a series >>> and I wasn't careful enough about sifting through the commits, likely >>> the jetlag wasn't helping. I'll aim to try harder, people can help by >>> pointing out patches they don't think should be going into the release >>> at this point... >> >> I read Sauls mail at the airport at 6:30 in the morning, which means >> that if I skipped sleeping I would have had a 16 hour window to >> object. For a bootloader patch that's not enough. Especially when the >> people who care about bootloaders are on spring break or attending >> collab. > > I don't expect people to wait for Saul's emails and then review patches > already on the mailing list. I don't expect that either, my point was that the window between the patch being posted and saul sending the "applied" email was 16 hours, half of it during PST sleepy time. That is waaaaay too short for a bootloader patch to get reviewed. Opening my mail at 6:30 I read the patch for the first time and thought "hmmm, this isn't right" and noticed Saul had already applied it. That will teach me for not checking my oe-core email for a day :) > Regardless, I'm rather torn in this case. Stefan's usecase is a valid > one and I don't see why we have to change the layout of DEPLOY_DIR > because of this one issue. I know of people doing some interesting > things with layouts and I don't really want to impose policy on that > which the change you propose does. Adding MACHINE to DEPLOY_DIR_IMAGE allows you to get rid of post processing scripts to rename foo-MACHINE to foo before consuming it, since you can now symlink safely to foo without stepping on other machines. > Having the symlink contain MACHINE > but not the name is rather odd. > > I also agree its late in the cycle for something which breaks build > scripts however :(. > > So I'm probably in favour of reverting from 1.2 but adding straight away > for 1.3 although I don't like letting 1.2 out with this issue present > really. Sounds good to me. regards, Koen ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-04-11 11:59 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-04-05 11:48 [PATCH] u-boot: add machine name to spl image name Stefan Herbrechtsmeier 2012-04-06 3:18 ` Saul Wold 2012-04-06 13:29 ` Koen Kooi 2012-04-10 13:41 ` Richard Purdie 2012-04-10 13:56 ` Koen Kooi 2012-04-10 14:19 ` Stefan Herbrechtsmeier 2012-04-10 14:22 ` Koen Kooi 2012-04-10 14:28 ` Martin Jansa 2012-04-11 11:36 ` Richard Purdie 2012-04-11 11:49 ` Koen Kooi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox