[coreboot] [PATCH 4/6] Geode GX2 cleanup patch

Peter Stuge peter at stuge.se
Sun Dec 26 06:08:36 CET 2010


Nils wrote:
> Remove wrong GX2 processor IIOC mode setting on CS5535 southbridge
> code and fix CIS mode comments.

Hm, please talk a little about this?


> +++ src/southbridge/amd/cs5535/early_setup.c	(working copy)
> @@ -107,15 +107,11 @@
..
> -	//Only do this if we are building for 5535
> -	msr.lo = 0x2;
> -	msr.hi = 0x0;
> -	wrmsr(VIP_GIO_MSR_SEL, msr);

This code is not added to another place by this patch. Is it simply
completely bogus, but harmless, for 5535? And harmful for GX2? Please
explain a little?


> +++ src/include/cpu/amd/gx2def.h	(working copy)
..
> +#define	FG_GIO_MSR_SEL		(MSR_FG + 0x2010)
..

> -#define VIP_GIO_MSR_SEL		(MSR_VIP + 0x2010)

(Why remove this?)


> +++ src/cpu/amd/model_gx2/cpureginit.c	(working copy)
..
> -	msrnum = MSR_FG + 0x10;
..
> +	msrnum = FG_GIO_MSR_SEL;

This is not the same MSR. Please explain? FG_GIO_MSR_SEL is defined
to MSR_FG + 0x2010 above, so this particular change changes which MSR
is being accessed. Is on purpose?


//Peter




More information about the coreboot mailing list