Uwe Hermann wrote:
> Factor out common CAR asm snippets.
> 
> This makes the CAR implementations a lot more readable, shorter and
> easier to follow, and also reduces the amount of uselessly duplicated code.
> 
> For example there are more than 12 open-coded "enable cache" instances
> spread all over the place (and 12 "disable cache" ones), multiple
> "enable mtrr", "save BIST", "restore BIST", etc. etc.
> 
> Signed-off-by: Uwe Hermann <[email protected]>

Nice! I would kill some blank lines too, but this is a really nice 
improvement either way!

Acked-by: Peter Stuge <[email protected]>

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to