[coreboot-gerrit] Change in coreboot[master]: arch/x86: Change power_res_dev_states[] to be static const * const

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/40935 )

Change subject: arch/x86: Change power_res_dev_states[] to be static const * const
......................................................................

arch/x86: Change power_res_dev_states[] to be static const * const

This change makes power_res_dev_states[] to be static const * const as
complained by Jenkins.

BUG=b:155428745

Change-Id: Ice2fff6ab3bcd72a059bc905b7462a681f2e6aaf
Signed-off-by: Furquan Shaikh 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40935
Tested-by: build bot (Jenkins) 
Reviewed-by: Aaron Durbin 
---
M src/arch/x86/acpi_device.c
M src/arch/x86/acpigen.c
M src/arch/x86/include/arch/acpigen.h
3 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Aaron Durbin: Looks good to me, approved

diff --git a/src/arch/x86/acpi_device.c b/src/arch/x86/acpi_device.c
index bef1a5d..9f63200 100644
--- a/src/arch/x86/acpi_device.c
+++ b/src/arch/x86/acpi_device.c
@@ -527,7 +527,7 @@
 /* PowerResource() with Enable and/or Reset control */
 void acpi_device_add_power_res(const struct acpi_power_res_params *params)
 {
-	static const char *power_res_dev_states[] = { "_PR0", "_PR3" };
+	static const char * const power_res_dev_states[] = { "_PR0", "_PR3" };
    unsigned int reset_gpio = params->reset_gpio ? params->reset_gpio->pins[0] : 0;
    unsigned int enable_gpio = params->enable_gpio ? params->enable_gpio->pins[0] : 0;
    unsigned int stop_gpio = params->stop_gpio ? params->stop_gpio->pins[0] : 0;
diff --git a/src/arch/x86/acpigen.c b/src/arch/x86/acpigen.c
index 34de2ba..dfc2a5a 100644
--- a/src/arch/x86/acpigen.c
+++ b/src/arch/x86/acpigen.c
@@ -1147,7 +1147,7 @@
  * PowerResource (name, level, order)
  */
 void acpigen_write_power_res(const char *name, uint8_t level, uint16_t order,
-			     const char *dev_states[], size_t dev_states_count)
+			     const char * const dev_states[], size_t dev_states_count)
 {
    size_t i;
    for (i = 0; i < dev_states_count; i++) {
diff --git a/src/arch/x86/include/arch/acpigen.h b/src/arch/x86/include/arch/acpigen.h
index 3339ce4..a3c4777 100644
--- a/src/arch/x86/include/arch/acpigen.h
+++ b/src/arch/x86/include/arch/acpigen.h
@@ -347,7 +347,7 @@
 void acpigen_write_irq(u16 mask);
 void acpigen_write_uuid(const char *uuid);
 void acpigen_write_power_res(const char *name, uint8_t level, uint16_t order,
-			     const char *dev_states[], size_t dev_states_count);
+			     const char * const dev_states[], size_t dev_states_count);
 void acpigen_write_sleep(uint64_t sleep_ms);
 void acpigen_write_store(void);
 void acpigen_write_store_ops(uint8_t src, uint8_t dst);