Albert, > -----Original Message----- > From: Albert ARIBAUD [mailto:[email protected]] > 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 > <[email protected]> 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. ----------------------------------------------------------------------------------- _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
