Dear Liu,

Am Montag, den 11.10.2010, 11:48 +0800 schrieb Liu Tao:

> the original code reads cpu ht speed from HT chain 0's register.
> the patch fix it to read the register from the chain which SB chip is on.
> 
> Signed-off-by: Liu Tao <[email protected]>

Thank you for your patch.

> Index: src/southbridge/amd/rs780/rs780_gfx.c
> ===================================================================
> --- src/southbridge/amd/rs780/rs780_gfx.c     (revision 5923)
> +++ src/southbridge/amd/rs780/rs780_gfx.c     (working copy)
> @@ -302,7 +302,7 @@
>       unsigned char * bpointer;
>       volatile u32 * GpuF0MMReg;
>       volatile u32 * pointer;
> -     int i;
> +     int i, sblk;
>       u16 command;
>       u32 value;
>       u16 deviceid, vendorid;
> @@ -453,9 +453,15 @@
>       vgainfo.usMinNBVoltage = 0;
>       vgainfo.usBootUpNBVoltage = 0x1a;
> 
> +     /* SB link */
> +     value = pci_read_config32(k8_f0, 0x64);
> +     sblk = (value >> 8) & 0x3;
> +     printk(BIOS_DEBUG, "SBLK = %d.\n", sblk);

Could you make that message more elaborate please. Like »SB link = …«?

> +
> +     /* HT speed */
>       value = pci_read_config32(nb_dev, 0xd0);
>       printk(BIOS_DEBUG, "NB HT speed = %x.\n", value);
> -     value = pci_read_config32(k8_f0, 0x88);
> +     value = pci_read_config32(k8_f0, 0x88 + (sblk * 0x20));
>       printk(BIOS_DEBUG, "CPU HT speed = %x.\n", value);
>       vgainfo.ulHTLinkFreq = 100 * 100; /* set HT speed. */


Thanks,

Paul

Attachment: signature.asc
Description: This is a digitally signed message part

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

Reply via email to