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.
and /home/rminnich/coreboot-v3/southbridge/intel/i82801gx/smbus.c:34: error: conflicting types for ?\226?\128?\152smbus_read_byte?\226?\128?\153 include/device/smbus.h:56: error: previous declaration of ?\226?\128?\152smbus_read_byte?\226?\128?\153 was here
we are working these. The second is much harder than it seems. It concerns whether we put i2c devices (i.e. DRAM spd SEEPROMS) in the dts.
I have a 8 month old patch to fix this, but the patch was vetoed back then. I could repost.
Regards, Carl-Daniel