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

[HAOUAS Elyes (Code Review)]

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

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

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

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

Change subject: nb/intel/haswell: Drop none reliable MCHBARx_{AND,OR} macros
......................................................................

nb/intel/haswell: Drop none reliable 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: Ib32f91de325a2291801807cdb2a887e18f08f102
Signed-off-by: Elyes HAOUAS 
---
M src/northbridge/intel/haswell/finalize.c
M src/northbridge/intel/haswell/haswell.h
M src/northbridge/intel/pineview/pineview.h
3 files changed, 7 insertions(+), 25 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/44458/4