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

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45517 )

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

Patch Set 37:

(1 comment)

Thanks for taking this on.

https://review.coreboot.org/c/coreboot/+/45517/37/src/northbridge/intel/comm... 
File src/northbridge/intel/common/mchbar_ops.h:

https://review.coreboot.org/c/coreboot/+/45517/37/src/northbridge/intel/comm... 
PS37, Line 27: (u8)
Are these casts already needed to get it through Jenkins? Otherwise, we
should drop them to have the compiler warn us about truncations that
actually matter.

An exception is the `(u8)~0` ofc., that is truncated on purpose. But we
could also just write it out, `0xff`?