[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Drop unreliable MCHBARx_{AND,OR} macros

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Nico Huber, Angel Pons, Arthur Heymans, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/44506

to look at the new patch set (#5).

Change subject: nb/intel/sandybridge: Drop unreliable MCHBARx_{AND,OR} macros
......................................................................

nb/intel/sandybridge: Drop unreliable MCHBARx_{AND,OR} macros

Use of MCHBARx_AND_OR gives an error:
 overflow in conversion from 'int' to 'u16' {aka 'volatile short unsigned int'} changes value from '(int)*(volatile u16 *)((unsigned int)((int)i * 1024) + 4275127072) & -65536 | 26214' to '26214' [-Werror=overflow]

Change-Id: I73f7f22dfa2440b3b44312e614a5d38ae1efcfc8
Signed-off-by: Elyes HAOUAS 
---
M src/northbridge/intel/sandybridge/early_init.c
M src/northbridge/intel/sandybridge/finalize.c
M src/northbridge/intel/sandybridge/northbridge.c
M src/northbridge/intel/sandybridge/raminit_common.c
M src/northbridge/intel/sandybridge/sandybridge.h
5 files changed, 75 insertions(+), 62 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/44506/5