From d059112eec64f79cf6a62197a96769cbd104252a Mon Sep 17 00:00:00 2001
From: Angel Pons <th3fanbus@gmail.com>
Date: Wed, 26 May 2021 14:24:06 +0200
Subject: [PATCH] nb/intel/x4x/rcven.c: Guard macro parameters

Add parentheses around macro parameters to avoid operation order issues.

Change-Id: I9528f3d6b221854fddd2db6d2b45c63bfdda0092
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/54953
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 src/northbridge/intel/x4x/rcven.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/northbridge/intel/x4x/rcven.c b/src/northbridge/intel/x4x/rcven.c
index d3b1b31d52..9e58ef33c0 100644
--- a/src/northbridge/intel/x4x/rcven.c
+++ b/src/northbridge/intel/x4x/rcven.c
@@ -10,7 +10,7 @@
 #define DQS_HIGH 1
 #define DQS_LOW 0
 
-#define RESET_CNTL(channel) (0x5d8 + channel * 0x400)
+#define RESET_CNTL(channel) (0x5d8 + (channel) * 0x400)
 
 struct rec_timing {
 	u8 medium;