Felix Held has uploaded this change for review.

View Change

nb/intel/sandybridge: improve indexed register helper macros

Replace the multiplications with corresponding shifts, so that it's
easier to see at which bit offsets the values get assigned.

Change-Id: I0b0d5172394ff65edfe57bdad474631938e58872
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
---
M src/northbridge/intel/sandybridge/sandybridge.h
1 file changed, 5 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/38577/1
diff --git a/src/northbridge/intel/sandybridge/sandybridge.h b/src/northbridge/intel/sandybridge/sandybridge.h
index f5c1e41..ffc1d9f 100644
--- a/src/northbridge/intel/sandybridge/sandybridge.h
+++ b/src/northbridge/intel/sandybridge/sandybridge.h
@@ -128,11 +128,11 @@
#define MCHBAR32_AND_OR(x, and, or) (MCHBAR32(x) = (MCHBAR32(x) & (and)) | (or))

/* Indexed register helper macros */
-#define Gz(r, z) ((r) + ((z) * 0x100))
-#define Ly(r, y) ((r) + ((y) * 4))
-#define Cx(r, x) ((r) + ((x) * 0x400))
-#define CxLy(r, x, y) ((r) + ((x) * 0x400) + ((y) * 4))
-#define GzLy(r, z, y) ((r) + ((z) * 0x100) + ((y) * 4))
+#define Gz(r, z) ((r) + ((z) << 8))
+#define Ly(r, y) ((r) + ((y) << 2))
+#define Cx(r, x) ((r) + ((x) << 10))
+#define CxLy(r, x, y) ((r) + ((x) << 10) + ((y) << 2))
+#define GzLy(r, z, y) ((r) + ((z) << 8) + ((y) << 2))

/* byte lane training register base addresses */
#define LANEBASE_B0 0x0000

To view, visit change 38577. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0b0d5172394ff65edfe57bdad474631938e58872
Gerrit-Change-Number: 38577
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Held <felix-coreboot@felixheld.de>
Gerrit-MessageType: newchange