On 14.11.2008 18:14, [EMAIL PROTECTED] 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 -- http://www.hailfinger.org/ -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
