[S] Change in coreboot[main]: sb/intel/bd82x6x: assign PCH SMBus controller ops in chipset devicetree

[Felix Held (Code Review)]

Attention is currently required from: Angel Pons.

Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/79168?usp=email )

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

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 
Change-Id: I3d3745ba5aefa30efbe705155d216aa7eadd26a7
---
M src/northbridge/intel/sandybridge/chipset.cb
M src/southbridge/intel/bd82x6x/smbus.c
2 files changed, 2 insertions(+), 10 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/79168/1

diff --git a/src/northbridge/intel/sandybridge/chipset.cb b/src/northbridge/intel/sandybridge/chipset.cb
index c7fbceb3..f11fd2f 100644
--- a/src/northbridge/intel/sandybridge/chipset.cb
+++ b/src/northbridge/intel/sandybridge/chipset.cb
@@ -39,7 +39,7 @@
    		device pci 1e.0 alias pci_bridge off end # PCI bridge
    		device pci 1f.0 alias lpc on ops bd82x6x_lpc_bridge_ops end
    		device pci 1f.2 alias sata1 off end      # SATA Controller 1
-			device pci 1f.3 alias smbus on end       # SMBus
+			device pci 1f.3 alias smbus on ops bd82x6x_smbus_ops end
    		device pci 1f.5 alias sata2 off end      # SATA Controller 2
    		device pci 1f.6 alias thermal off end    # Thermal
    	end
diff --git a/src/southbridge/intel/bd82x6x/smbus.c b/src/southbridge/intel/bd82x6x/smbus.c
index 0cb6d00..d2ca247 100644
--- a/src/southbridge/intel/bd82x6x/smbus.c
+++ b/src/southbridge/intel/bd82x6x/smbus.c
@@ -28,7 +28,7 @@
    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 @@
    .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,
-};