On 14.11.2008 19:04, ron minnich wrote:
On Fri, Nov 14, 2008 at 9:56 AM, ron minnich rminnich@gmail.com wrote:
On Fri, Nov 14, 2008 at 9:53 AM, Stefan Reinauer stepan@coresystems.de wrote:
Carl-Daniel Hailfinger wrote:
On 14.11.2008 18:14, svn@coreboot.org wrote:
Author: rminnich New Revision: 1026
/home/rminnich/coreboot-v3/build/coreboot.initram_partiallylinked.o: section .data.rel.ro.local: dual_channel_slew_group_lookup.3242 single_channel_slew_group_lookup.3243
Basic rule: If you want to have arrays of pointers in initram, you lose. Pointers are not relocatable by definition. const is not going to help you there.
Hm. This is bad. 'nother regression in v3 that wasn't in v2.
it's not acceptable as a rule. We have to fix it one way or another. Why gcc is marking it as writeable is a puzzle but points to a bug somewhere.
for now I'm fixing it with a section attribute.
Compilation may work, but RAMinit will fail during runtime if you do that.
Regards, Carl-Daniel