Martin Roth has uploaded this change for review. ( https://review.coreboot.org/24930
Change subject: soc/amd/stoneyridge: Add ST/CZ SMBus device id ......................................................................
soc/amd/stoneyridge: Add ST/CZ SMBus device id
The SMBus PCI device ID for Stoney wasn't updated when the code was pulled over from hudson. This means that the IOAPIC wasn't being initialized in coreboot.
BUG=b:74070580 TEST=Boot Grunt, see IOAPIC init messages in console.
Change-Id: Ida5d3f3592488694681300d79444c1e26fff6a1a Signed-off-by: Martin Roth martinroth@chromium.org --- M src/include/device/pci_ids.h M src/soc/amd/stoneyridge/sm.c 2 files changed, 4 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/24930/1
diff --git a/src/include/device/pci_ids.h b/src/include/device/pci_ids.h index 16ca1a9..cbdaca1 100644 --- a/src/include/device/pci_ids.h +++ b/src/include/device/pci_ids.h @@ -442,6 +442,8 @@ #define PCI_DEVICE_ID_AMD_CZ_USB_0 0x7907 #define PCI_DEVICE_ID_AMD_CZ_USB_1 0x7908 #define PCI_DEVICE_ID_AMD_CZ_USB3_0 0x7914 +#define PCI_DEVICE_ID_AMD_CZ_SMBUS 0x790B +
#define PCI_VENDOR_ID_VLSI 0x1004 #define PCI_DEVICE_ID_VLSI_82C592 0x0005 diff --git a/src/soc/amd/stoneyridge/sm.c b/src/soc/amd/stoneyridge/sm.c index 9456cbf..ca593ce 100644 --- a/src/soc/amd/stoneyridge/sm.c +++ b/src/soc/amd/stoneyridge/sm.c @@ -120,8 +120,9 @@ .ops_pci = &lops_pci, .ops_smbus_bus = &lops_smbus_bus, }; + static const struct pci_driver smbus_driver __pci_driver = { .ops = &smbus_ops, .vendor = PCI_VENDOR_ID_AMD, - .device = PCI_DEVICE_ID_AMD_SB900_SM, + .device = PCI_DEVICE_ID_AMD_CZ_SMBUS, };