Patch committed in r1071, thanks.<br><br>On Wed, Dec 10, 2008 at 5:11 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="Ih2E3d">On 10.12.2008 23:02, Myles Watson wrote:<br>
> On Wed, Dec 10, 2008 at 2:57 PM, Carl-Daniel Hailfinger<br>
> <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>> wrote:<br>
><br>
><br>
</div><div class="Ih2E3d">>> You're welcome. I admit that off-by-one errors are pretty hard to spot<br>
>> and your code really looked correct at the first glance.<br>
>> We have an old saying at my university about such bugs: "Programmers are<br>
>> either off by one or by a factor of two." It happened to me often<br>
>> enough. ;-)<br>
>><br>
>><br>
</div><div class="Ih2E3d">>> On 10.12.2008 22:51, Corey Osgood wrote:<br>
>><br>
</div><div class="Ih2E3d">>>> Per Segher's email I'd prefer to go with<br>
>>> Carl-Daniel's original suggestion of using 0x100000000.<br>
>>><br>
><br>
> Sounds good.<br>
><br>
><br>
> Even though the Kconfig solution proposed was ugly, could we think of a<br>
> different one that would still live in Kconfig.  It seems a lot nicer to<br>
> catch it during configuration than during the build.</div></blockquote><div class="Ih2E3d"><br>Yes and no, IMO. Yes in that, well, it gets caught earlier. No because there's no real way (that I know of...) to explain through Kconfig that the reason you can't set a larger ROM size is because of the CAR area, which could leave some users confused.<br>

<br>-Corey<br>
</div></div><br>