commit aa090cb6ea696d9b6a2869e16c53bd8b14a0e7d1
author Aaron Durbin <adurbin@chromium.org> Wed Sep 13 16:01:52 2017 -0600
committer Aaron Durbin <adurbin@chromium.org> Thu Sep 14 14:34:27 2017 +0000
tree d605b4aabb1c9001049c1500c8b10c5e35219755
parent d07f377872d1d229a8a9cc1d4a2c66f535e95fe3

device: acpi_name() should take a const struct device

There's no reason to mutate the struct device when determining
the ACPI name for a device. Adjust the function pointer
signature and the respective implementations to use const
struct device.

Change-Id: If5e1f4de36a53646616581b01f47c4e86822c42e
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/21527
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>

src/southbridge/intel/bd82x6x/azalia.c

diff --git a/src/southbridge/intel/bd82x6x/azalia.c b/src/southbridge/intel/bd82x6x/azalia.c
index 0e974ba..9c8eaf6 100644
--- a/src/southbridge/intel/bd82x6x/azalia.c
+++ b/src/southbridge/intel/bd82x6x/azalia.c
@@ -331,7 +331,7 @@
 	pci_write_config8(dev, 0x43, reg8);
 }
 
-static const char *azalia_acpi_name(device_t dev)
+static const char *azalia_acpi_name(const struct device *dev)
 {
 	return "HDEF";
 }

src/southbridge/intel/bd82x6x/lpc.c

diff --git a/src/southbridge/intel/bd82x6x/lpc.c b/src/southbridge/intel/bd82x6x/lpc.c
index 901f71b..3f72219 100644
--- a/src/southbridge/intel/bd82x6x/lpc.c
+++ b/src/southbridge/intel/bd82x6x/lpc.c
@@ -814,7 +814,7 @@
 	fadt->x_gpe1_blk.addrh = 0x0;
 }
 
-static const char *lpc_acpi_name(device_t dev)
+static const char *lpc_acpi_name(const struct device *dev)
 {
 	return "LPCB";
 }

src/southbridge/intel/bd82x6x/pcie.c

diff --git a/src/southbridge/intel/bd82x6x/pcie.c b/src/southbridge/intel/bd82x6x/pcie.c
index a3eaba9..34b7694 100644
--- a/src/southbridge/intel/bd82x6x/pcie.c
+++ b/src/southbridge/intel/bd82x6x/pcie.c
@@ -285,7 +285,7 @@
 	pch_pcie_pm_late(dev);
 }
 
-static const char *pch_pcie_acpi_name(device_t dev)
+static const char *pch_pcie_acpi_name(const struct device *dev)
 {
 	ASSERT(dev);
 

src/southbridge/intel/bd82x6x/sata.c

diff --git a/src/southbridge/intel/bd82x6x/sata.c b/src/southbridge/intel/bd82x6x/sata.c
index ac4945d..d4268da 100644
--- a/src/southbridge/intel/bd82x6x/sata.c
+++ b/src/southbridge/intel/bd82x6x/sata.c
@@ -244,7 +244,7 @@
 	}
 }
 
-static const char *sata_acpi_name(device_t dev)
+static const char *sata_acpi_name(const struct device *dev)
 {
 	return "SATA";
 }

src/southbridge/intel/bd82x6x/smbus.c

diff --git a/src/southbridge/intel/bd82x6x/smbus.c b/src/southbridge/intel/bd82x6x/smbus.c
index 6cbdef9..33908905 100644
--- a/src/southbridge/intel/bd82x6x/smbus.c
+++ b/src/southbridge/intel/bd82x6x/smbus.c
@@ -100,7 +100,7 @@
 	res = pci_get_resource(dev, PCI_BASE_ADDRESS_0);
 }
 
-static const char *smbus_acpi_name(device_t dev)
+static const char *smbus_acpi_name(const struct device *dev)
 {
 	return "SBUS";
 }

src/southbridge/intel/bd82x6x/usb_ehci.c

diff --git a/src/southbridge/intel/bd82x6x/usb_ehci.c b/src/southbridge/intel/bd82x6x/usb_ehci.c
index 42d7a83..90b29c4 100644
--- a/src/southbridge/intel/bd82x6x/usb_ehci.c
+++ b/src/southbridge/intel/bd82x6x/usb_ehci.c
@@ -94,7 +94,7 @@
 	pci_write_config8(dev, 0x80, access_cntl);
 }
 
-static const char *usb_ehci_acpi_name(device_t dev)
+static const char *usb_ehci_acpi_name(const struct device *dev)
 {
 	switch (dev->path.pci.devfn) {
 	case PCI_DEVFN(0x1a, 0):

src/southbridge/intel/bd82x6x/usb_xhci.c

diff --git a/src/southbridge/intel/bd82x6x/usb_xhci.c b/src/southbridge/intel/bd82x6x/usb_xhci.c
index 1aeb864..6acc63b 100644
--- a/src/southbridge/intel/bd82x6x/usb_xhci.c
+++ b/src/southbridge/intel/bd82x6x/usb_xhci.c
@@ -51,7 +51,7 @@
 	printk(BIOS_DEBUG, "done.\n");
 }
 
-static const char *xhci_acpi_name(device_t dev)
+static const char *xhci_acpi_name(const struct device *dev)
 {
 	return "XHC";
 }