[coreboot-gerrit] New patch to review for coreboot: AMD binaryPI: Drop commented code with bad PCI IDs

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Sun Nov 20 02:57:11 CET 2016


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17508

-gerrit

commit c75c4eae83e9ca96b4c7ab64d96f7497f0adde2a
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Sat Nov 19 22:38:13 2016 +0200

    AMD binaryPI: Drop commented code with bad PCI IDs
    
    There is mismatch of VENDOR_ID_AMD with DEVICE_ID_ATI, also
    the device IDs have not been defined.
    
    Change-Id: I3076cb08e3181e7f86de38deb18f1661f037bc38
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/southbridge/amd/pi/hudson/usb.c | 25 -------------------------
 1 file changed, 25 deletions(-)

diff --git a/src/southbridge/amd/pi/hudson/usb.c b/src/southbridge/amd/pi/hudson/usb.c
index 7398d25..9eb35cc 100644
--- a/src/southbridge/amd/pi/hudson/usb.c
+++ b/src/southbridge/amd/pi/hudson/usb.c
@@ -54,28 +54,3 @@ static const struct pci_driver usb_0_driver __pci_driver = {
 	.vendor = PCI_VENDOR_ID_AMD,
 	.devices = pci_device_ids,
 };
-
-/*
-static struct device_operations usb_ops2 = {
-	.read_resources = pci_ehci_read_resources,
-	.set_resources = pci_dev_set_resources,
-	.enable_resources = pci_dev_enable_resources,
-	.init = usb_init2,
-	.scan_bus = 0,
-	.ops_pci = &lops_pci,
-};
-*/
-/*
-static const struct pci_driver usb_5_driver __pci_driver = {
-	.ops = &usb_ops2,
-	.vendor = PCI_VENDOR_ID_AMD,
-	.device = PCI_DEVICE_ID_ATI_HUDSON_USB_18_2,
-};
-*/
-/*
- * static const struct pci_driver usb_5_driver __pci_driver = {
- * 	.ops = &usb_ops2,
- * 	.vendor = PCI_VENDOR_ID_AMD,
- * 	.device = PCI_DEVICE_ID_ATI_HUDSON_USB_19_2,
- * };
- */



More information about the coreboot-gerrit mailing list