[coreboot-gerrit] Change in coreboot[master]: nb/intel: Introduce MCHBAR accessor functions

[HAOUAS Elyes (Code Review)]

Attention is currently required from: Nico Huber, Angel Pons.
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45517 )

Change subject: nb/intel: Introduce MCHBAR accessor functions
......................................................................

Patch Set 47:

(3 comments)

File src/northbridge/intel/common/mchbar_ops.h:

https://review.coreboot.org/c/coreboot/+/45517/comment/2aebb0fe_e71ea79b 
PS37, Line 8: (u8 *)DEFAULT_MCHBAR + x)
> Macro safety, please. Also, should we avoid pointer arithmetics? […]

Done

https://review.coreboot.org/c/coreboot/+/45517/comment/2c5eca93_c0c42136 
PS37, Line 27: (u8)
> Sorry, I mixed that up because of the changed name. It's still inverted […]

Ack

File src/northbridge/intel/gm45/gm45.h:

https://review.coreboot.org/c/coreboot/+/45517/comment/67d0ad62_004ce50c 
PS42, Line 170: #include "memmap.h"
> duplicated.

Done