sb/intel/bd82x6x: assign PCH SMBus controller ops in chipset devicetree

Since the SMBus controller in the PCH is always on the same device
function, the device operations can be statically assigned in the
devicetree and there's no need to bind the SMBus device operations to
the PCI device during runtime via a list of PCI IDs.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I3d3745ba5aefa30efbe705155d216aa7eadd26a7
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79168
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

1 files changed, 1 insertions, 9 deletions

diff --git a/src/southbridge/intel/bd82x6x/smbus.c b/src/southbridge/intel/bd82x6x/smbus.c
index 0cb6d0029e..d2ca24767b 100644
--- a/src/southbridge/intel/bd82x6x/smbus.c
+++ b/src/southbridge/intel/bd82x6x/smbus.c
@@ -28,7 +28,7 @@ static const char *smbus_acpi_name(const struct device *dev)
 	return "SBUS";
 }
 
-static struct device_operations smbus_ops = {
+struct device_operations bd82x6x_smbus_ops = {
 	.read_resources		= smbus_read_resources,
 	.set_resources		= pci_dev_set_resources,
 	.enable_resources	= pci_dev_enable_resources,
@@ -38,11 +38,3 @@ static struct device_operations smbus_ops = {
 	.ops_pci		= &pci_dev_ops_pci,
 	.acpi_name		= smbus_acpi_name,
 };
-
-static const unsigned short pci_device_ids[] = { 0x1c22, 0x1e22, 0 };
-
-static const struct pci_driver pch_smbus __pci_driver = {
-	.ops	 = &smbus_ops,
-	.vendor	 = PCI_VID_INTEL,
-	.devices = pci_device_ids,
-};