[coreboot-gerrit] Change in ...coreboot[master]: src/mb/Kconfig: Fix PCI subsystem IDs

[Kyösti Mälkki (Code Review)]

Hello HAOUAS Elyes,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/32107

to review the following change.

Change subject: src/mb/Kconfig: Fix PCI subsystem IDs
......................................................................

src/mb/Kconfig: Fix PCI subsystem IDs

Change-Id: I72bba8406eea4a264e36cc9bcf467cf5cfbed379
Signed-off-by: Elyes HAOUAS 
Signed-off-by: Kyösti Mälkki 
---
M src/mainboard/gigabyte/ga-h61m-s2pv/Kconfig
M src/mainboard/lenovo/t520/Kconfig
M src/mainboard/lenovo/t530/Kconfig
M util/autoport/bd82x6x.go
4 files changed, 0 insertions(+), 30 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/32107/1

diff --git a/src/mainboard/gigabyte/ga-h61m-s2pv/Kconfig b/src/mainboard/gigabyte/ga-h61m-s2pv/Kconfig
index 5fba47a..5e46365 100644
--- a/src/mainboard/gigabyte/ga-h61m-s2pv/Kconfig
+++ b/src/mainboard/gigabyte/ga-h61m-s2pv/Kconfig
@@ -40,14 +40,6 @@
    string
    default "GA-H61M-S2PV"

-config MAINBOARD_PCI_SUBSYSTEM_DEVICE_ID
-	hex
-	default 0x5001
-
-config MAINBOARD_PCI_SUBSYSTEM_VENDOR_ID
-	hex
-	default 0x1458
-
 config MAX_CPUS
    int
    default 8
diff --git a/src/mainboard/lenovo/t520/Kconfig b/src/mainboard/lenovo/t520/Kconfig
index c414083..06f296b 100644
--- a/src/mainboard/lenovo/t520/Kconfig
+++ b/src/mainboard/lenovo/t520/Kconfig
@@ -65,14 +65,6 @@
    string
    default "pci8086,0126.rom"

-config MAINBOARD_PCI_SUBSYSTEM_VENDOR_ID
-	hex
-	default 0x17aa
-
-config MAINBOARD_PCI_SUBSYSTEM_DEVICE_ID
-	hex
-	default 0x21cf
-
 config ONBOARD_VGA_IS_PRIMARY
    bool
    default y
diff --git a/src/mainboard/lenovo/t530/Kconfig b/src/mainboard/lenovo/t530/Kconfig
index 14f9fb2..b36460d 100644
--- a/src/mainboard/lenovo/t530/Kconfig
+++ b/src/mainboard/lenovo/t530/Kconfig
@@ -62,14 +62,6 @@
    default "pci8086,0106.rom" if BOARD_LENOVO_T530
    default "pci8086,0166.rom" if BOARD_LENOVO_W530

-config MAINBOARD_PCI_SUBSYSTEM_VENDOR_ID
-	hex
-	default 0x17aa
-
-config MAINBOARD_PCI_SUBSYSTEM_DEVICE_ID
-	hex
-	default 0x21fa
-
 config ONBOARD_VGA_IS_PRIMARY
    bool
    default y
diff --git a/util/autoport/bd82x6x.go b/util/autoport/bd82x6x.go
index 265cb6f..a97ff1d 100644
--- a/util/autoport/bd82x6x.go
+++ b/util/autoport/bd82x6x.go
@@ -181,12 +181,6 @@
    }
    KconfigComment["DRAM_RESET_GATE_GPIO"] = "FIXME: check this"

-	/* Not strictly speaking correct. These subsys/subvendor referer to PCI devices.
-	   But most systems don't have any of those. But the config needs to be set
-	   nevertheless. So set it to southbridge subsys/subvendor.  */
-	KconfigHex["MAINBOARD_PCI_SUBSYSTEM_VENDOR_ID"] = uint32(GetLE16(addr.ConfigDump[0x2c:0x2e]))
-	KconfigHex["MAINBOARD_PCI_SUBSYSTEM_DEVICE_ID"] = uint32(GetLE16(addr.ConfigDump[0x2e:0x30]))
-
    ich9GetFlashSize(ctx)

DSDTDefines = append(DSDTDefines,