commit 30c3f91d33a8308db80a730c6df27553d8fdb072
author Duncan Laurie <dlaurie@google.com> Fri Oct 09 04:48:53 2020 +0000
committer Duncan Laurie <dlaurie@chromium.org> Wed Oct 21 22:24:19 2020 +0000
tree d74f0519bf2046c9d2546c442536f284818e4a10
parent 36858208e6e06fdda2698e9a66a8e27f6d9acd24

acpigen: Make gpio set/get arguments const

The 'struct acpi_gpio' arguments passed to acpigen functions are
not modified so they can be made const, which allows drivers to
also use a const pointer.

Signed-off-by: Duncan Laurie <dlaurie@google.com>
Change-Id: I59e9c19e7bfdca275230776497767ddc7f6c52db
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46257
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/acpi/acpigen.c

diff --git a/src/acpi/acpigen.c b/src/acpi/acpigen.c
index 0ed3708..e2fe2cf 100644
--- a/src/acpi/acpigen.c
+++ b/src/acpi/acpigen.c
@@ -1865,7 +1865,7 @@
  *
  * Returns 0 on success and -1 on error.
  */
-int acpigen_enable_tx_gpio(struct acpi_gpio *gpio)
+int acpigen_enable_tx_gpio(const struct acpi_gpio *gpio)
 {
 	if (gpio->active_low)
 		return acpigen_soc_clear_tx_gpio(gpio->pins[0]);
@@ -1873,7 +1873,7 @@
 		return acpigen_soc_set_tx_gpio(gpio->pins[0]);
 }
 
-int acpigen_disable_tx_gpio(struct acpi_gpio *gpio)
+int acpigen_disable_tx_gpio(const struct acpi_gpio *gpio)
 {
 	if (gpio->active_low)
 		return acpigen_soc_set_tx_gpio(gpio->pins[0]);
@@ -1881,7 +1881,7 @@
 		return acpigen_soc_clear_tx_gpio(gpio->pins[0]);
 }
 
-void acpigen_get_rx_gpio(struct acpi_gpio *gpio)
+void acpigen_get_rx_gpio(const struct acpi_gpio *gpio)
 {
 	acpigen_soc_read_rx_gpio(gpio->pins[0]);
 
@@ -1889,7 +1889,7 @@
 		acpigen_write_xor(LOCAL0_OP, 1, LOCAL0_OP);
 }
 
-void acpigen_get_tx_gpio(struct acpi_gpio *gpio)
+void acpigen_get_tx_gpio(const struct acpi_gpio *gpio)
 {
 	acpigen_soc_get_tx_gpio(gpio->pins[0]);