diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2017-09-13 16:01:52 -0600 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2017-09-14 14:34:27 +0000 |
| commit | aa090cb6ea696d9b6a2869e16c53bd8b14a0e7d1 (patch) | |
| tree | d605b4aabb1c9001049c1500c8b10c5e35219755 /src/southbridge/amd | |
| parent | d07f377872d1d229a8a9cc1d4a2c66f535e95fe3 (diff) | |
| download | coreboot-aa090cb6ea696d9b6a2869e16c53bd8b14a0e7d1.tar.xz | |
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>
Diffstat (limited to 'src/southbridge/amd')
| -rw-r--r-- | src/southbridge/amd/cimx/sb800/late.c | 2 | ||||
| -rw-r--r-- | src/southbridge/amd/pi/hudson/lpc.c | 2 | ||||
| -rw-r--r-- | src/southbridge/amd/sb700/lpc.c | 3 |
3 files changed, 4 insertions, 3 deletions
diff --git a/src/southbridge/amd/cimx/sb800/late.c b/src/southbridge/amd/cimx/sb800/late.c index 58d3a3432a..ed4f8a5ab3 100644 --- a/src/southbridge/amd/cimx/sb800/late.c +++ b/src/southbridge/amd/cimx/sb800/late.c @@ -143,7 +143,7 @@ unsigned long acpi_fill_mcfg(unsigned long current) return current; } -static const char *lpc_acpi_name(struct device *dev) +static const char *lpc_acpi_name(const struct device *dev) { if (dev->path.type != DEVICE_PATH_PCI) return NULL; diff --git a/src/southbridge/amd/pi/hudson/lpc.c b/src/southbridge/amd/pi/hudson/lpc.c index 7a3914a9bd..282f17c1f1 100644 --- a/src/southbridge/amd/pi/hudson/lpc.c +++ b/src/southbridge/amd/pi/hudson/lpc.c @@ -339,7 +339,7 @@ unsigned long acpi_fill_mcfg(unsigned long current) return current; } -static const char *lpc_acpi_name(struct device *dev) +static const char *lpc_acpi_name(const struct device *dev) { if (dev->path.type != DEVICE_PATH_PCI) return NULL; diff --git a/src/southbridge/amd/sb700/lpc.c b/src/southbridge/amd/sb700/lpc.c index 2943b96a94..643a5a1300 100644 --- a/src/southbridge/amd/sb700/lpc.c +++ b/src/southbridge/amd/sb700/lpc.c @@ -252,7 +252,8 @@ static void southbridge_acpi_fill_ssdt_generator(device_t device) { amd_generate_powernow(ACPI_CPU_CONTROL, 6, 1); } -static const char *lpc_acpi_name(struct device *dev) { +static const char *lpc_acpi_name(const struct device *dev) +{ if (dev->path.type != DEVICE_PATH_PCI) return NULL; |