* [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 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.