Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33157
Change subject: mb/google/link: Make iodefs array const ......................................................................
mb/google/link: Make iodefs array const
This array is fixed during compile time and is not modified during program execution, so we can make it const.
Change-Id: I3b4a71e1763b6d7e09859691f3a8d829f5f04577 Signed-off-by: Jacob Garber jgarber1@ualberta.ca --- M src/mainboard/google/link/i915.c M src/mainboard/google/link/i915io.c 2 files changed, 5 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/33157/1
diff --git a/src/mainboard/google/link/i915.c b/src/mainboard/google/link/i915.c index 12d424c..955ae4f 100644 --- a/src/mainboard/google/link/i915.c +++ b/src/mainboard/google/link/i915.c @@ -132,8 +132,8 @@ return microseconds(globalstart, rdtscll()); }
-extern struct iodef iodefs[]; -extern int niodefs; +extern const struct iodef iodefs[]; +extern const int niodefs;
static int i915_init_done = 0;
@@ -154,7 +154,7 @@ static int run(int index) { int i, prev = 0; - struct iodef *id, *lastidread = NULL; + const struct iodef *id, *lastidread = NULL; unsigned long u, t; if (index >= niodefs) return index; diff --git a/src/mainboard/google/link/i915io.c b/src/mainboard/google/link/i915io.c index 5ebb42d..dea78d0 100644 --- a/src/mainboard/google/link/i915io.c +++ b/src/mainboard/google/link/i915io.c @@ -17,7 +17,7 @@ #include <stdlib.h> #include "i915io.h"
-struct iodef iodefs[] = { +const struct iodef iodefs[] = { {V, 0}, {W, 1, "", PCH_GMBUS0, 0x00000000, 0}, {R, 1, "", PP_ON_DELAYS, ( /* T2 */ 0x0 << 16) | ( /* T5 */ 0x0 << 0) | 0x00000000, 0}, @@ -352,4 +352,4 @@ {I,}, };
-int niodefs = ARRAY_SIZE(iodefs); +const int niodefs = ARRAY_SIZE(iodefs);