[coreboot] [PATCH]es: Use CBFS on amd/serengeti_cheetah and iwill/dk8_htx

[Patrick Georgi]

Am 07.06.2009 15:10, schrieb Carl-Daniel Hailfinger:
> Hi Patrick,
>
> On 06.06.2009 15:14, Patrick Georgi wrote:
>    
>> "failover"-infrastructure to CBFS.
>>
>> 2. Set ROM_IMAGE_SIZE so that
>> FAILOVER_SIZE + ROM_IMAGE_SIZE = XIP_ROM_SIZE
>> to avoid problems with XIP.
>>
>>      
>
> Actually, this is a big problem for me. Which XIP_ROM_SIZE setting is
> honored? The one in src/mainboard/$VENDOR/$BOARD/Config.lb or the one in
> src/mainboard/$VENDOR/$BOARD/Options.lb or the one in
> targets/$VENDOR/$BOARD/Config.lb or targets/$VENDOR/$BOARD/Config-abuild.lb?
>    
Whatever is effective ;-) (helpful, I know)

In my opinion targets/*/*/Config*lb shouldn't define any XIP_* values 
except for failover (and maybe not even that)
And in src/mainboard, it should also only be in one of the files, to not 
confuse users. (again imho)
> I have a script which does all conversions except ROM_IMAGE_SIZE. Once
> my question above is answered, I should be able to fix my script and
> post it.
> What bothers me is that my script changed more in the iwill/dk8_htx case
> than your patch.
>    
I wrote the instructions from looking at the patches and not the other 
way around, so the "canonical" solution might be different. But there's 
more than one valid way to do it. As long as it builds, it's a step in 
the right direction, in my opinion. There will be more changes to make 
the rom layout more space efficient etc, that should simplify and 
disambiguate things.

I'm on vacation the next week and thus offline.


Regards,
Patrick
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090607/806a98fe/attachment.html>