[coreboot] v3 patch: include
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Tue Aug 12 04:15:34 CEST 2008
On 12.08.2008 04:10, Carl-Daniel Hailfinger wrote:
> On 12.08.2008 02:00, ron minnich wrote:
>
>> Make some things (die, mem*, resourcemap code, option code) SHARED.
>> The option code is tricky as it is used by standalone code. If you include
>> that file and you are standalone, you now have to define STANDALONE
>> (is there a better way?)
>>
>> Change the cpuid to be a 24-byte string instead of 3 u32s.
>>
>> Make the CPUID usage PIC-safe by not using %ebx.
>>
>> Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
>>
>>
>
> Please keep in mind that every shared function bloats the image.
>
> I assume you have build tested this change on all targets. It has the
> potential to blow up on Geode.
>
I forgot the Ack. It's simply too late for me.
If it builds on all targets (except the K8 one)
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list