[coreboot] [PATCH] v3: convert fake SPD to struct spd_entry

ron minnich rminnich at gmail.com
Tue Feb 26 20:11:32 CET 2008


On Tue, Feb 26, 2008 at 11:04 AM, Uwe Hermann <uwe at hermann-uwe.de> wrote:
> On Tue, Feb 26, 2008 at 07:53:16PM +0100, Carl-Daniel Hailfinger wrote:
>  > Convert all boards using fake SPD entries to struct spd_entry, thereby
>  > making sure we return 0xff for nonexisiting entries and shrinking the
>  > data structure by 85%.
>
>  Can you elaborate? 85% less in RAM or in ROM? How did you measure this?

due to the big sparse struct. It's not a huge deal after compression :-)
>
>
>
>  > As a bonus, the various initram.c for boards with fake SPD are now
>  > almost identical.
>
>  Nice! (next step: generic code in lib/, drop per-board code)


yeah but be careful.

There are lots of special cases. I think we could close out most of
them as follows:
provide an array called valid_spd_device which contains a 0-terminate
list of valid SPDs.

Then the function above can, first:
1. for loop to find out if spd is valid, if not, return 0xff
2. for loop that is currently there.

I can provide sample code if this is not clear.

But this is still cool. I will test on alix1c tonight and report back.

ron




More information about the coreboot mailing list