[coreboot] [PATCH try 4] Reduce duplicate definition in CAR code.

Kevin O'Connor kevin at koconnor.net
Thu Oct 7 14:58:17 CEST 2010


On Thu, Oct 07, 2010 at 04:35:04AM -0700, Warren Turkal wrote:
> I added names for the AMD registers and separated each .long onto it's
> own line to avoid any bugs of the type the caldani mentioned.
> 
> Comments? Acks? :)

Just my $0.02 - I like how you've replaced the magic numbers with
names.  I don't like how you've moved the lists to another file.

My suggestion would be to replace all the magic numbers in the
assembler files.  Then once that's complete, send patches with your
proposal to change the assembler layout.

BTW, the list concept doesn't make much sense anyway - at least on
Via, there is no need to clear the fixed mtrrs, and you don't need a
list to clear the variable mtrrs (a simple iterator would suffice).

-Kevin




More information about the coreboot mailing list