[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