Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/49805 )
Change subject: cpu/intel/haswell: Constify ACPI c-state arrays ......................................................................
cpu/intel/haswell: Constify ACPI c-state arrays
Change-Id: I5538d8279392238e59aba99ade4b5fe13f250ca8 Signed-off-by: Angel Pons th3fanbus@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/49805 Reviewed-by: Nico Huber nico.h@gmx.de Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/cpu/intel/haswell/acpi.c 1 file changed, 4 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved
diff --git a/src/cpu/intel/haswell/acpi.c b/src/cpu/intel/haswell/acpi.c index dfbda45..8eec704 100644 --- a/src/cpu/intel/haswell/acpi.c +++ b/src/cpu/intel/haswell/acpi.c @@ -87,19 +87,19 @@ }, };
-static int cstate_set_s0ix[3] = { +static const int cstate_set_s0ix[3] = { C_STATE_C1E, C_STATE_C7S_LONG_LAT, C_STATE_C10, };
-static int cstate_set_lp[3] = { +static const int cstate_set_lp[3] = { C_STATE_C1E, C_STATE_C3, C_STATE_C7S_LONG_LAT, };
-static int cstate_set_trad[3] = { +static const int cstate_set_trad[3] = { C_STATE_C1, C_STATE_C3, C_STATE_C6_LONG_LAT, @@ -182,7 +182,7 @@ { acpi_cstate_t acpi_cstate_map[3] = {0};
- int *acpi_cstates; + const int *acpi_cstates;
if (is_s0ix_enabled()) acpi_cstates = cstate_set_s0ix;