[coreboot] r774 - coreboot-v3/southbridge/amd/amd8111

svn at coreboot.org svn at coreboot.org
Sat Aug 16 01:37:29 CEST 2008


Author: rminnich
Date: 2008-08-16 01:37:29 +0200 (Sat, 16 Aug 2008)
New Revision: 774

Modified:
   coreboot-v3/southbridge/amd/amd8111/amd8111_smbus.h
   coreboot-v3/southbridge/amd/amd8111/stage1_smbus.c
Log:
Small typos.

Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
Acked-by: Ronald G. Minnich <rminnich at gmail.com>



Modified: coreboot-v3/southbridge/amd/amd8111/amd8111_smbus.h
===================================================================
--- coreboot-v3/southbridge/amd/amd8111/amd8111_smbus.h	2008-08-15 22:04:31 UTC (rev 773)
+++ coreboot-v3/southbridge/amd/amd8111/amd8111_smbus.h	2008-08-15 23:37:29 UTC (rev 774)
@@ -1,4 +1,5 @@
 #include <device/smbus_def.h>
+#include <io.h>
 
 #define SMBGSTATUS 0xe0
 #define SMBGCTL    0xe2

Modified: coreboot-v3/southbridge/amd/amd8111/stage1_smbus.c
===================================================================
--- coreboot-v3/southbridge/amd/amd8111/stage1_smbus.c	2008-08-15 22:04:31 UTC (rev 773)
+++ coreboot-v3/southbridge/amd/amd8111/stage1_smbus.c	2008-08-15 23:37:29 UTC (rev 774)
@@ -37,12 +37,11 @@
 
 void enable_smbus(void)
 {
-	u32 bdf;
+	u32 dev;
 	u8 enable;
 
-	/* this 746b is the ACPI device. This is from original code. It's weird however. */
-	pci_locate_device_on_bus(0, PCI_VENDOR_ID_AMD, 0x746b, &dev);
-	if (dev == PCI_DEV_INVALID) {
+	/* Yes, the ACPI device controls the SMBUS.  */
+	if (!pci_conf1_find_on_bus(0, PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_8111_ACPI, &dev)){
 		die("SMBUS controller not found\r\n");
 	}
 





More information about the coreboot mailing list