[coreboot] [PATCH try 2] Reduce duplicate definition in CAR code.
ron minnich
rminnich at gmail.com
Wed Oct 6 00:18:09 CEST 2010
It's proven dangerous in the past to cross mtrr settings across vendors.
Which is what you are doing. Then somebody patches, e.g.,
cpu/x86/mtrr.h for some fix to via, and we find out a year later it is
not right for some flavor of AMD. MTRRs have been a rolling headache
for 10 years now.
Sure they should all be the same. Sometimes there are weird issues.
So what I'd prefer, personally: leave the settings in each vendor
file: amd, via, whatever, don't make the common settings in
cpu/x86/mtrr.h. But use your nice macros to set those up.
ron
More information about the coreboot
mailing list