Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38036 )
Change subject: nb/intel/sandybridge: Add a bunch of MCHBAR defines ......................................................................
Patch Set 6:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38036/4/src/northbridge/intel/sandy... File src/northbridge/intel/sandybridge/early_init.c:
https://review.coreboot.org/c/coreboot/+/38036/4/src/northbridge/intel/sandy... PS4, Line 38: 0x5404
VTD1_BASE + 4
Done
https://review.coreboot.org/c/coreboot/+/38036/4/src/northbridge/intel/sandy... PS4, Line 40: 0x5414
VTD2_BASE + 4
Done
https://review.coreboot.org/c/coreboot/+/38036/3/src/northbridge/intel/sandy... File src/northbridge/intel/sandybridge/sandybridge.h:
https://review.coreboot.org/c/coreboot/+/38036/3/src/northbridge/intel/sandy... PS3, Line 222: #define IOSAV_B0_BW_SERROR_C_C0 0x4140 /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B1_BW_SERROR_C_C0 0x4144 /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B2_BW_SERROR_C_C0 0x4148 /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B3_BW_SERROR_C_C0 0x414c /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B4_BW_SERROR_C_C0 0x4150 /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B5_BW_SERROR_C_C0 0x4154 /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B6_BW_SERROR_C_C0 0x4158 /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B7_BW_SERROR_C_C0 0x415c /* IOSAV Bytelane 0 Bit-wise error */ : #define IOSAV_B8_BW_SERROR_C_C0 0x4160 /* IOSAV Bytelane 0 Bit-wise error */
Copy-pasta error in the comments. Meh, I will replace this with indexed macros anyway.
Done