> 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





-- 
linuxbios mailing list
linuxbios@linuxbios.org
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to