Change in ...coreboot[master]: mb/google/link: Make iodefs array const

[Jacob Garber (Code Review)]

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 
---
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);

[Jacob Garber (Code Review)]

Jacob Garber has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/33157 )

Change subject: mb/google/link: Make iodefs array const
......................................................................

Abandoned

Superseded by https://review.coreboot.org/c/coreboot/+/33128