[LinuxBIOS] [PATCH] AMD Barcelona(family 10) support 1 of 4
Myles Watson
myles at pel.cs.byu.edu
Thu Dec 13 18:37:15 CET 2007
> Please focus most
> attention on the few places where changes were made to preexisting
> files.
>Index: LinuxBIOSv2/src/config/Options.lb
>===================================================================
>--- LinuxBIOSv2.orig/src/config/Options.lb 2007-12-12
>11:03:38.000000000 -0700
>+++ LinuxBIOSv2/src/config/Options.lb 2007-12-12 14:16:22.000000000 ->0700
>@@ -291,6 +291,11 @@
> export always
> comment "Use data cache as temporary RAM if possible"
> end
>+define CAR_FAM10
>+ default 0
>+ export always
>+ comment "AMD family 10 CAR need set more"
>+end
This could be a little more clear. Did you mean "needs more setup" or
something else?
>Index: LinuxBIOSv2/src/cpu/amd/car/cache_as_ram.inc
>Index: LinuxBIOSv2/src/cpu/amd/microcode/microcode.c
It looks like most of the changes are white space in these files. It would
be a lot easier to understand the changes if you submitted the patch as
1. a white space and license patch
2. a CAR_FAM10 patch
I also didn't understand why the type change from uint32_t to u32 was
important.
Myles
More information about the coreboot
mailing list