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

Peter Stuge peter at stuge.se
Thu Feb 28 00:45:23 CET 2008


On Tue, Feb 26, 2008 at 05:55:02PM -0800, ron minnich wrote:
> As we add more things to dts, we should add more code generation.

Agreed. I don't think we can generate initram though.


> We do not want to have a situation where we're doing this at
> runtime: spdval = value("/dram/spd/tpr");
> because there is lots of build time checking we lose (e.g. somebody
> specifies a totally wacky 32-bit number for what is a u8 value).

I still think contents should be checked at build time. Actually,
for static SPD it would be an improvement to do some validation at
build time rather than failing in initram.


> I like the dts but I'm not sure we want every single thing in it.

Not every thing, but I would like it to abstract the board.


//Peter




More information about the coreboot mailing list