[coreboot] [PATCH]Refactor SPDROM access
Patrick Georgi
patrick at georgi-clan.de
Sat Nov 20 00:31:14 CET 2010
Hi,
attached patch cleans up SPDROM access and gets rid of the DIMMx
definitions in romstage.cs (in favor of a single set of them in spd.h).
Of special note is that amd/cs5535 and amd/cs5536 used different values
for DIMMx: These were simply bitshifted. Their only use is in
smbus_read_byte (pre-ram variant, which doesn't seem to be used for
anything else), so I adapted their version to do the same bitshift the
other implementations do. That way, they can reuse the same DIMMx
defines, too.
A later change would be to move the 90% of spd_read_byte implementations
in the boards to a unified one (weak function) that can be overridden by
boards that really behave differently (eg. those with fixed RAM without
SPD ROMs). The single-DIMM special case could be accomodated with a
Kconfig variable then.
The patch is untested, so please review carefully.
Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 20101120-1-spdrom-access-refactoring.diff
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20101120/a71eafc7/attachment.ksh>
More information about the coreboot
mailing list