[coreboot] r1026 - in coreboot-v3: mainboard/kontron/986lcd-m southbridge/intel/i82801gx

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Nov 14 18:28:45 CET 2008


On 14.11.2008 18:14, svn at 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

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list