* [U-Boot] [ARM] Rollback *again* -- 5 patches
@ 2013-04-11 15:25 Albert ARIBAUD
2013-04-11 15:49 ` Stephen Warren
2013-04-11 16:30 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches Albert ARIBAUD
0 siblings, 2 replies; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-11 15:25 UTC (permalink / raw)
To: u-boot
Hello,
Older toolchain and PPC incompatibilities appeared in the last two
additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
back from abbecf4c87 to ae74b65987. This removes
abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
(HIDDEN not known by most binutils 2.22 or older ones)
aad17a2 ARM: fix CONFIG_SPL_MAX_SIZE semantics
72c1601 smdk5250, snow: fix CONFIG_SPL_MAX_SIZE semantics
55e46fc da850evm, da850_am18xxevm: fix CONFIG_SPL_MAX_SIZE semantics
0502854 cam_enc_4xx: fix CONFIG_SPL_MAX_SIZE semantics
(PPC builds fail because --pad-to receives an expr, not a constant)
Prafulla (and any other impacted ARM tree custodians), be sure to rebase
as necessary.
Stephen, that is an occasion to add your proposal to aad17a2.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 15:25 [U-Boot] [ARM] Rollback *again* -- 5 patches Albert ARIBAUD
@ 2013-04-11 15:49 ` Stephen Warren
2013-04-11 16:37 ` Albert ARIBAUD
2013-04-11 16:30 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches Albert ARIBAUD
1 sibling, 1 reply; 14+ messages in thread
From: Stephen Warren @ 2013-04-11 15:49 UTC (permalink / raw)
To: u-boot
On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> Hello,
>
> Older toolchain and PPC incompatibilities appeared in the last two
> additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
> back from abbecf4c87 to ae74b65987. This removes
>
> abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
...
> Stephen, that is an occasion to add your proposal to aad17a2.
Tom Warren as the Tegra maintainer should be taking care of this. I am
simply pointing out that the currently proposed changes to the SPL size
checking don't fulfil the requirements for Tegra. I don't expect to have
to fix the patches.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches...
2013-04-11 15:25 [U-Boot] [ARM] Rollback *again* -- 5 patches Albert ARIBAUD
2013-04-11 15:49 ` Stephen Warren
@ 2013-04-11 16:30 ` Albert ARIBAUD
2013-04-11 16:33 ` Benoît Thébaudeau
2013-04-12 8:27 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches... -- FIXED Albert ARIBAUD
1 sibling, 2 replies; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-11 16:30 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Thu, 11 Apr 2013 17:25:34 +0200, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hello,
>
> Older toolchain and PPC incompatibilities appeared in the last two
> additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
> back from abbecf4c87 to ae74b65987. This removes
>
> abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
>
> (HIDDEN not known by most binutils 2.22 or older ones)
>
> aad17a2 ARM: fix CONFIG_SPL_MAX_SIZE semantics
> 72c1601 smdk5250, snow: fix CONFIG_SPL_MAX_SIZE semantics
> 55e46fc da850evm, da850_am18xxevm: fix CONFIG_SPL_MAX_SIZE semantics
> 0502854 cam_enc_4xx: fix CONFIG_SPL_MAX_SIZE semantics
>
> (PPC builds fail because --pad-to receives an expr, not a constant)
Actually, the failure was not due to this series, but to patch
16/30 of Beno?t's series. In agreement with Tom, rolling further back,
removing the recent TI PR merge and all of Beno?t's series.
Beno?t, can you fix 16/30 wrt PPC building? If you provide branches, I
or Tom can do tests.
Once v12 is ready, I'll apply it back, then the TI PR, then the
__bss_end patch, then the SPL max size series, this one list in order
to give us time to choose how overall SPL text+data+rodata+BSS
size testing should be done.
This still stands:
> Prafulla (and any other impacted ARM tree custodians), be sure to rebase
> as necessary.
>
> Stephen, that is an occasion to add your proposal to aad17a2.
>
> Amicalement,
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches...
2013-04-11 16:30 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches Albert ARIBAUD
@ 2013-04-11 16:33 ` Benoît Thébaudeau
2013-04-11 16:48 ` Albert ARIBAUD
2013-04-11 16:48 ` Tom Rini
2013-04-12 8:27 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches... -- FIXED Albert ARIBAUD
1 sibling, 2 replies; 14+ messages in thread
From: Benoît Thébaudeau @ 2013-04-11 16:33 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Thursday, April 11, 2013 6:30:58 PM, Albert ARIBAUD wrote:
> Hi Albert,
>
> On Thu, 11 Apr 2013 17:25:34 +0200, Albert ARIBAUD
> <albert.u.boot@aribaud.net> wrote:
>
> > Hello,
> >
> > Older toolchain and PPC incompatibilities appeared in the last two
> > additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
> > back from abbecf4c87 to ae74b65987. This removes
> >
> > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> >
> > (HIDDEN not known by most binutils 2.22 or older ones)
> >
> > aad17a2 ARM: fix CONFIG_SPL_MAX_SIZE semantics
> > 72c1601 smdk5250, snow: fix CONFIG_SPL_MAX_SIZE semantics
> > 55e46fc da850evm, da850_am18xxevm: fix CONFIG_SPL_MAX_SIZE semantics
> > 0502854 cam_enc_4xx: fix CONFIG_SPL_MAX_SIZE semantics
> >
> > (PPC builds fail because --pad-to receives an expr, not a constant)
>
> Actually, the failure was not due to this series, but to patch
> 16/30 of Beno?t's series. In agreement with Tom, rolling further back,
> removing the recent TI PR merge and all of Beno?t's series.
>
> Beno?t, can you fix 16/30 wrt PPC building? If you provide branches, I
> or Tom can do tests.
>
> Once v12 is ready, I'll apply it back, then the TI PR, then the
> __bss_end patch, then the SPL max size series, this one list in order
> to give us time to choose how overall SPL text+data+rodata+BSS
> size testing should be done.
OK, I'll do that. I need to know:
- which board fails,
- where to find the toolchain binaries ready to use for PPC (Ubuntu package?),
- if all other arches have been build-tested in order to avoid new issues
popping out later.
Best regards,
Beno?t
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 15:49 ` Stephen Warren
@ 2013-04-11 16:37 ` Albert ARIBAUD
2013-04-11 17:59 ` Tom Warren
0 siblings, 1 reply; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-11 16:37 UTC (permalink / raw)
To: u-boot
Hi Stephen,
On Thu, 11 Apr 2013 09:49:45 -0600, Stephen Warren
<swarren@wwwdotorg.org> wrote:
> On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> > Hello,
> >
> > Older toolchain and PPC incompatibilities appeared in the last two
> > additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
> > back from abbecf4c87 to ae74b65987. This removes
> >
> > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> ...
> > Stephen, that is an occasion to add your proposal to aad17a2.
>
> Tom Warren as the Tegra maintainer should be taking care of this. I am
> simply pointing out that the currently proposed changes to the SPL size
> checking don't fulfil the requirements for Tegra. I don't expect to have
> to fix the patches.
I did not mean for you to do it, only that there was an opportunity to
do it. :)
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches...
2013-04-11 16:33 ` Benoît Thébaudeau
@ 2013-04-11 16:48 ` Albert ARIBAUD
2013-04-11 17:03 ` Tom Rini
2013-04-11 16:48 ` Tom Rini
1 sibling, 1 reply; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-11 16:48 UTC (permalink / raw)
To: u-boot
Hi Beno?t,
On Thu, 11 Apr 2013 18:33:35 +0200 (CEST), Beno?t Th?baudeau
<benoit.thebaudeau@advansee.com> wrote:
> Hi Albert,
>
> On Thursday, April 11, 2013 6:30:58 PM, Albert ARIBAUD wrote:
> > Hi Albert,
> >
> > On Thu, 11 Apr 2013 17:25:34 +0200, Albert ARIBAUD
> > <albert.u.boot@aribaud.net> wrote:
> >
> > > Hello,
> > >
> > > Older toolchain and PPC incompatibilities appeared in the last two
> > > additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
> > > back from abbecf4c87 to ae74b65987. This removes
> > >
> > > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> > >
> > > (HIDDEN not known by most binutils 2.22 or older ones)
> > >
> > > aad17a2 ARM: fix CONFIG_SPL_MAX_SIZE semantics
> > > 72c1601 smdk5250, snow: fix CONFIG_SPL_MAX_SIZE semantics
> > > 55e46fc da850evm, da850_am18xxevm: fix CONFIG_SPL_MAX_SIZE semantics
> > > 0502854 cam_enc_4xx: fix CONFIG_SPL_MAX_SIZE semantics
> > >
> > > (PPC builds fail because --pad-to receives an expr, not a constant)
> >
> > Actually, the failure was not due to this series, but to patch
> > 16/30 of Beno?t's series. In agreement with Tom, rolling further back,
> > removing the recent TI PR merge and all of Beno?t's series.
> >
> > Beno?t, can you fix 16/30 wrt PPC building? If you provide branches, I
> > or Tom can do tests.
> >
> > Once v12 is ready, I'll apply it back, then the TI PR, then the
> > __bss_end patch, then the SPL max size series, this one list in order
> > to give us time to choose how overall SPL text+data+rodata+BSS
> > size testing should be done.
>
> OK, I'll do that. I need to know:
> - which board fails,
P1021RDB-PC_NAND
> - where to find the toolchain binaries ready to use for PPC (Ubuntu package?),
Toolchain is ELDK 5.2.1 for powerpc (installs in /opt, need to set
PATH=...:/opt/eldk-5.2.1/powerpc/sysroots/i686-eldk-linux/usr/bin/powerpc-linux,
ARCH=powerpc, and CROSS_COMPILE=powerpc-linux-
> - if all other arches have been build-tested in order to avoid new issues
> popping out later.
Tom?
> Best regards,
> Beno?t
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches...
2013-04-11 16:33 ` Benoît Thébaudeau
2013-04-11 16:48 ` Albert ARIBAUD
@ 2013-04-11 16:48 ` Tom Rini
1 sibling, 0 replies; 14+ messages in thread
From: Tom Rini @ 2013-04-11 16:48 UTC (permalink / raw)
To: u-boot
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 04/11/2013 12:33 PM, Beno?t Th?baudeau wrote:
> Hi Albert,
>
> On Thursday, April 11, 2013 6:30:58 PM, Albert ARIBAUD wrote:
>> Hi Albert,
>>
>> On Thu, 11 Apr 2013 17:25:34 +0200, Albert ARIBAUD
>> <albert.u.boot@aribaud.net> wrote:
>>
>>> Hello,
>>>
>>> Older toolchain and PPC incompatibilities appeared in the last
>>> two additions to ToT ARM. To avoid non-bisectability, ToT has
>>> been rolled back from abbecf4c87 to ae74b65987. This removes
>>>
>>> abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
>>>
>>> (HIDDEN not known by most binutils 2.22 or older ones)
>>>
>>> aad17a2 ARM: fix CONFIG_SPL_MAX_SIZE semantics 72c1601
>>> smdk5250, snow: fix CONFIG_SPL_MAX_SIZE semantics 55e46fc
>>> da850evm, da850_am18xxevm: fix CONFIG_SPL_MAX_SIZE semantics
>>> 0502854 cam_enc_4xx: fix CONFIG_SPL_MAX_SIZE semantics
>>>
>>> (PPC builds fail because --pad-to receives an expr, not a
>>> constant)
>>
>> Actually, the failure was not due to this series, but to patch
>> 16/30 of Beno?t's series. In agreement with Tom, rolling further
>> back, removing the recent TI PR merge and all of Beno?t's
>> series.
>>
>> Beno?t, can you fix 16/30 wrt PPC building? If you provide
>> branches, I or Tom can do tests.
>>
>> Once v12 is ready, I'll apply it back, then the TI PR, then the
>> __bss_end patch, then the SPL max size series, this one list in
>> order to give us time to choose how overall SPL
>> text+data+rodata+BSS size testing should be done.
>
> OK, I'll do that. I need to know: - which board fails, - where to
> find the toolchain binaries ready to use for PPC (Ubuntu
> package?), - if all other arches have been build-tested in order to
> avoid new issues popping out later.
P1021RDB-PC_NAND fails and I use ELDK 5.2 for a toolchain for PowerPC.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJRZuluAAoJENk4IS6UOR1WxeQP/2X5/dNDz2DyMnd6qaSxZMEG
2wlMRkGjOe8vA47R0NzqS5KyUMhwvMZAhebk798lRjqkY/t5cdFy1sXEnWI3kod3
z4j456bpoQpfbctsaqHw0nCDU/HKBW53BDsFIRTA++2obeZ3Ywmj+TQxkefAgBJY
UoyL33yZKC2M+CjeCKMaHJlmBYEkhthLxYn84zedoN/L8RZi66DXQ5QuGitRWZPL
DfewModX/KNvjyjyJNca4fGymT0PtWvB3gS39Kju9qcF9iSp/aTlla5/xdVx1gY5
FvnXheabmp1iNkyKD/783HExBkbVg99aMWJAo8ARc9HfbZwgY8Dlji7hdVWGXPYG
+RWgPwIp4PdKYTZnC5BK/sAHzWAVMv9iOxzkunxSbla9ULMysh1kPD9x+PVECo57
gJ9Eutui9q5NSLSUF/cIx2NXB9977ScYiFvy5+DX7MxytZvjOOQKor99qA9SqSYJ
kiOybsKtlemfR+wAxPy2hGvxf5CUX/Yc6NPk16x74kQ3bZIy+0/Z8hWkGpRTqDY6
jW8jDtDv90YwnDYVEsQBEoCyBjr/5g2a3sOuTUBhYszMTzC+eJVn8nVaMqyiWPpX
DBGMHC0o9pHJva1jVMf7AH7Q/uD4oUVmZRiSNpZ1znct1WU6D3ik/pTbsG+v3wIE
Qa2PbeHT2Y8Pe3uFzFLN
=2cE2
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches...
2013-04-11 16:48 ` Albert ARIBAUD
@ 2013-04-11 17:03 ` Tom Rini
0 siblings, 0 replies; 14+ messages in thread
From: Tom Rini @ 2013-04-11 17:03 UTC (permalink / raw)
To: u-boot
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 04/11/2013 12:48 PM, Albert ARIBAUD wrote:
> Hi Beno?t,
>
> On Thu, 11 Apr 2013 18:33:35 +0200 (CEST), Beno?t Th?baudeau
> <benoit.thebaudeau@advansee.com> wrote:
[snip]
>> - if all other arches have been build-tested in order to avoid
>> new issues popping out later.
>
> Tom?
Everything else seemed fine.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJRZuzjAAoJENk4IS6UOR1Wf3cP/2/Bt+79bqmElu+fMJZvhitM
fEMvblZYxKvY3u5+t9dUWi8yo2VnrMDHpyXcaMqcBzByf6mOm2T729UmGuYKAKUl
GBqUFcgHjxpycjDMM13OcVppzP8K6QFpI4SHEm7F67mEmqIm48NqHvH3oBCRwVIV
Se9dOIFQYlKV0on3JfSCj3deQB5+yPQ98/4mZbybzJnBfwfiX8oU7Q37JeWaRf9W
lZu8OMdHQhWpyvP6zHTT9MAqZKfuSWT2Ie1HQNj9vhBWwZpk20Gk6v9+sZ36FFGI
qYDaBrAKv2NzcZG5OYiNehb/aeX+iSnmJhcBkCElKqbp6xZW23YzQxPZwTl6c0ot
w/XvLF52p1bHUiXfu0MatFI6ik1T5ON0H+XVjEoVKIo73+OBB646NtikmwMvGnxl
bO+68lFLBMMe6+8L4qxhbDwwnNGpznbczcKVon5V45cTKiMSgRqkWdRSodPl98jp
UcZXjMk7zdsLakSBvgiUFjZ4bkYrV8MJJSZcA9jI7PWAuGEblA2zKdMCiRbhFhmR
kXLYE5WmcVgUc0A6g4oJQSd1hv87iGbRAbMcTvXV3Il72aquytLUqksTOQ6dajQ8
GKhnr//qmWuzbxhmn7F16Gv1Zs+e2t43awdjYW96fh1jCIgbLFgKjxwBMWTJSvSP
+k/rWf2Mik2uLd+wspVE
=9sN6
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 16:37 ` Albert ARIBAUD
@ 2013-04-11 17:59 ` Tom Warren
2013-04-11 18:01 ` Albert ARIBAUD
0 siblings, 1 reply; 14+ messages in thread
From: Tom Warren @ 2013-04-11 17:59 UTC (permalink / raw)
To: u-boot
Albert,
> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> Sent: Thursday, April 11, 2013 9:38 AM
> To: Stephen Warren
> Cc: U-Boot; Prafulla Wadaskar; Tom Rini; Tom Warren
> Subject: Re: [ARM] Rollback *again* -- 5 patches
>
> Hi Stephen,
>
> On Thu, 11 Apr 2013 09:49:45 -0600, Stephen Warren
> <swarren@wwwdotorg.org> wrote:
>
> > On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> > > Hello,
> > >
> > > Older toolchain and PPC incompatibilities appeared in the last two
> > > additions to ToT ARM. To avoid non-bisectability, ToT has been
> > > rolled back from abbecf4c87 to ae74b65987. This removes
> > >
> > > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> > ...
> > > Stephen, that is an occasion to add your proposal to aad17a2.
> >
> > Tom Warren as the Tegra maintainer should be taking care of this. I am
> > simply pointing out that the currently proposed changes to the SPL
> > size checking don't fulfil the requirements for Tegra. I don't expect
> > to have to fix the patches.
>
> I did not mean for you to do it, only that there was an opportunity to do it. :)
Why should I be fixing something I didn't break, and that worked fine before your change?
As Stephen has pointed out, Tegra uses the whole enchilada (text/rodata/data/bss) for the SPL portion. Any change you add should account for that. Even if we are unique in that use/requirement, it still needs to work correctly before and after your patch.
Tom
>
> Amicalement,
> --
> Albert.
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information. Any unauthorized review, use, disclosure or distribution
is prohibited. If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 17:59 ` Tom Warren
@ 2013-04-11 18:01 ` Albert ARIBAUD
2013-04-11 18:04 ` Tom Warren
0 siblings, 1 reply; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-11 18:01 UTC (permalink / raw)
To: u-boot
Hi Tom,
On Thu, 11 Apr 2013 10:59:29 -0700, Tom Warren <TWarren@nvidia.com>
wrote:
> Albert,
>
> > -----Original Message-----
> > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> > Sent: Thursday, April 11, 2013 9:38 AM
> > To: Stephen Warren
> > Cc: U-Boot; Prafulla Wadaskar; Tom Rini; Tom Warren
> > Subject: Re: [ARM] Rollback *again* -- 5 patches
> >
> > Hi Stephen,
> >
> > On Thu, 11 Apr 2013 09:49:45 -0600, Stephen Warren
> > <swarren@wwwdotorg.org> wrote:
> >
> > > On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> > > > Hello,
> > > >
> > > > Older toolchain and PPC incompatibilities appeared in the last two
> > > > additions to ToT ARM. To avoid non-bisectability, ToT has been
> > > > rolled back from abbecf4c87 to ae74b65987. This removes
> > > >
> > > > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> > > ...
> > > > Stephen, that is an occasion to add your proposal to aad17a2.
> > >
> > > Tom Warren as the Tegra maintainer should be taking care of this. I am
> > > simply pointing out that the currently proposed changes to the SPL
> > > size checking don't fulfil the requirements for Tegra. I don't expect
> > > to have to fix the patches.
> >
> > I did not mean for you to do it, only that there was an opportunity to do it. :)
>
> Why should I be fixing something I didn't break, and that worked fine before your change?
Er...
Was "I did not mean for you to do it" unclear?
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 18:01 ` Albert ARIBAUD
@ 2013-04-11 18:04 ` Tom Warren
2013-04-11 18:13 ` Albert ARIBAUD
0 siblings, 1 reply; 14+ messages in thread
From: Tom Warren @ 2013-04-11 18:04 UTC (permalink / raw)
To: u-boot
Albert,
On Thu, Apr 11, 2013 at 11:01 AM, Albert ARIBAUD
<albert.u.boot@aribaud.net>wrote:
> Hi Tom,
>
> On Thu, 11 Apr 2013 10:59:29 -0700, Tom Warren <TWarren@nvidia.com>
> wrote:
>
> > Albert,
> >
> > > -----Original Message-----
> > > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> > > Sent: Thursday, April 11, 2013 9:38 AM
> > > To: Stephen Warren
> > > Cc: U-Boot; Prafulla Wadaskar; Tom Rini; Tom Warren
> > > Subject: Re: [ARM] Rollback *again* -- 5 patches
> > >
> > > Hi Stephen,
> > >
> > > On Thu, 11 Apr 2013 09:49:45 -0600, Stephen Warren
> > > <swarren@wwwdotorg.org> wrote:
> > >
> > > > On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> > > > > Hello,
> > > > >
> > > > > Older toolchain and PPC incompatibilities appeared in the last two
> > > > > additions to ToT ARM. To avoid non-bisectability, ToT has been
> > > > > rolled back from abbecf4c87 to ae74b65987. This removes
> > > > >
> > > > > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> > > > ...
> > > > > Stephen, that is an occasion to add your proposal to aad17a2.
> > > >
> > > > Tom Warren as the Tegra maintainer should be taking care of this. I
> am
> > > > simply pointing out that the currently proposed changes to the SPL
> > > > size checking don't fulfil the requirements for Tegra. I don't expect
> > > > to have to fix the patches.
> > >
> > > I did not mean for you to do it, only that there was an opportunity to
> do it. :)
> >
> > Why should I be fixing something I didn't break, and that worked fine
> before your change?
>
> Er...
>
> Was "I did not mean for you to do it" unclear?
>
Yes, it was unclear.
Tom
>
> Amicalement,
> --
> Albert.
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 18:04 ` Tom Warren
@ 2013-04-11 18:13 ` Albert ARIBAUD
2013-04-12 6:17 ` Albert ARIBAUD
0 siblings, 1 reply; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-11 18:13 UTC (permalink / raw)
To: u-boot
Hi Tom,
On Thu, 11 Apr 2013 11:04:44 -0700, Tom Warren
<twarren.nvidia@gmail.com> wrote:
> Albert,
>
>
> On Thu, Apr 11, 2013 at 11:01 AM, Albert ARIBAUD
> <albert.u.boot@aribaud.net>wrote:
>
> > Hi Tom,
> >
> > On Thu, 11 Apr 2013 10:59:29 -0700, Tom Warren <TWarren@nvidia.com>
> > wrote:
> >
> > > Albert,
> > >
> > > > -----Original Message-----
> > > > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> > > > Sent: Thursday, April 11, 2013 9:38 AM
> > > > To: Stephen Warren
> > > > Cc: U-Boot; Prafulla Wadaskar; Tom Rini; Tom Warren
> > > > Subject: Re: [ARM] Rollback *again* -- 5 patches
> > > >
> > > > Hi Stephen,
> > > >
> > > > On Thu, 11 Apr 2013 09:49:45 -0600, Stephen Warren
> > > > <swarren@wwwdotorg.org> wrote:
> > > >
> > > > > On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> > > > > > Hello,
> > > > > >
> > > > > > Older toolchain and PPC incompatibilities appeared in the last two
> > > > > > additions to ToT ARM. To avoid non-bisectability, ToT has been
> > > > > > rolled back from abbecf4c87 to ae74b65987. This removes
> > > > > >
> > > > > > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> > > > > ...
> > > > > > Stephen, that is an occasion to add your proposal to aad17a2.
> > > > >
> > > > > Tom Warren as the Tegra maintainer should be taking care of this. I
> > am
> > > > > simply pointing out that the currently proposed changes to the SPL
> > > > > size checking don't fulfil the requirements for Tegra. I don't expect
> > > > > to have to fix the patches.
> > > >
> > > > I did not mean for you to do it, only that there was an opportunity to
> > do it. :)
> > >
> > > Why should I be fixing something I didn't break, and that worked fine
> > before your change?
> >
> > Er...
> >
> > Was "I did not mean for you to do it" unclear?
> >
> Yes, it was unclear.
Apologies, then. I only meant to say that we had time to discuss and
incorporate your solution in the SPL max size patch series since it had
just had to be "unapplied".
> Tom
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches
2013-04-11 18:13 ` Albert ARIBAUD
@ 2013-04-12 6:17 ` Albert ARIBAUD
0 siblings, 0 replies; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-12 6:17 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Thu, 11 Apr 2013 20:13:16 +0200, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Tom,
>
> On Thu, 11 Apr 2013 11:04:44 -0700, Tom Warren
> <twarren.nvidia@gmail.com> wrote:
>
> > Albert,
> >
> >
> > On Thu, Apr 11, 2013 at 11:01 AM, Albert ARIBAUD
> > <albert.u.boot@aribaud.net>wrote:
> >
> > > Hi Tom,
> > >
> > > On Thu, 11 Apr 2013 10:59:29 -0700, Tom Warren <TWarren@nvidia.com>
> > > wrote:
> > >
> > > > Albert,
> > > >
> > > > > -----Original Message-----
> > > > > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> > > > > Sent: Thursday, April 11, 2013 9:38 AM
> > > > > To: Stephen Warren
> > > > > Cc: U-Boot; Prafulla Wadaskar; Tom Rini; Tom Warren
> > > > > Subject: Re: [ARM] Rollback *again* -- 5 patches
> > > > >
> > > > > Hi Stephen,
> > > > >
> > > > > On Thu, 11 Apr 2013 09:49:45 -0600, Stephen Warren
> > > > > <swarren@wwwdotorg.org> wrote:
> > > > >
> > > > > > On 04/11/2013 09:25 AM, Albert ARIBAUD wrote:
> > > > > > > Hello,
> > > > > > >
> > > > > > > Older toolchain and PPC incompatibilities appeared in the last two
> > > > > > > additions to ToT ARM. To avoid non-bisectability, ToT has been
> > > > > > > rolled back from abbecf4c87 to ae74b65987. This removes
> > > > > > >
> > > > > > > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> > > > > > ...
> > > > > > > Stephen, that is an occasion to add your proposal to aad17a2.
> > > > > >
> > > > > > Tom Warren as the Tegra maintainer should be taking care of this. I
> > > am
> > > > > > simply pointing out that the currently proposed changes to the SPL
> > > > > > size checking don't fulfil the requirements for Tegra. I don't expect
> > > > > > to have to fix the patches.
> > > > >
> > > > > I did not mean for you to do it, only that there was an opportunity to
> > > do it. :)
> > > >
> > > > Why should I be fixing something I didn't break, and that worked fine
> > > before your change?
> > >
> > > Er...
> > >
> > > Was "I did not mean for you to do it" unclear?
> > >
> > Yes, it was unclear.
>
> Apologies, then. I only meant to say that we had time to discuss and
> incorporate your solution in the SPL max size patch series since it had
*Stephen's* solution.
Sorry, yesterday was kind of hectic.
> just had to be "unapplied".
>
> > Tom
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches... -- FIXED
2013-04-11 16:30 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches Albert ARIBAUD
2013-04-11 16:33 ` Benoît Thébaudeau
@ 2013-04-12 8:27 ` Albert ARIBAUD
1 sibling, 0 replies; 14+ messages in thread
From: Albert ARIBAUD @ 2013-04-12 8:27 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Thu, 11 Apr 2013 18:30:58 +0200, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Albert,
>
> On Thu, 11 Apr 2013 17:25:34 +0200, Albert ARIBAUD
> <albert.u.boot@aribaud.net> wrote:
>
> > Hello,
> >
> > Older toolchain and PPC incompatibilities appeared in the last two
> > additions to ToT ARM. To avoid non-bisectability, ToT has been rolled
> > back from abbecf4c87 to ae74b65987. This removes
> >
> > abbecf4 ARM: Fix __bss_start and __bss_end in linker scripts
> >
> > (HIDDEN not known by most binutils 2.22 or older ones)
> >
> > aad17a2 ARM: fix CONFIG_SPL_MAX_SIZE semantics
> > 72c1601 smdk5250, snow: fix CONFIG_SPL_MAX_SIZE semantics
> > 55e46fc da850evm, da850_am18xxevm: fix CONFIG_SPL_MAX_SIZE semantics
> > 0502854 cam_enc_4xx: fix CONFIG_SPL_MAX_SIZE semantics
> >
> > (PPC builds fail because --pad-to receives an expr, not a constant)
>
> Actually, the failure was not due to this series, but to patch
> 16/30 of Beno?t's series. In agreement with Tom, rolling further back,
> removing the recent TI PR merge and all of Beno?t's series.
Status:
- Beno?t's series was fixed and build-tested against ARM and PPC.
- Fabio's watchdog removal patch was reapplied.
- Tom's TI PR was re-applied.
ARM Tot (c97b6df1ae92679d67f94c1cfef51323782a506d) can now be used
again for basing PRs and patch submissions.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2013-04-12 8:27 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-11 15:25 [U-Boot] [ARM] Rollback *again* -- 5 patches Albert ARIBAUD
2013-04-11 15:49 ` Stephen Warren
2013-04-11 16:37 ` Albert ARIBAUD
2013-04-11 17:59 ` Tom Warren
2013-04-11 18:01 ` Albert ARIBAUD
2013-04-11 18:04 ` Tom Warren
2013-04-11 18:13 ` Albert ARIBAUD
2013-04-12 6:17 ` Albert ARIBAUD
2013-04-11 16:30 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches Albert ARIBAUD
2013-04-11 16:33 ` Benoît Thébaudeau
2013-04-11 16:48 ` Albert ARIBAUD
2013-04-11 17:03 ` Tom Rini
2013-04-11 16:48 ` Tom Rini
2013-04-12 8:27 ` [U-Boot] [ARM] Rollback *again* -- 5 patches PLUS 31 patches... -- FIXED Albert ARIBAUD
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox