Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47164 )
Change subject: sb/intel/common: Add code to configure who decodes IO 0x80 ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/47164/2/src/southbridge/intel/commo... File src/southbridge/intel/common/post.h:
https://review.coreboot.org/c/coreboot/+/47164/2/src/southbridge/intel/commo... PS2, Line 6: Nit, coding style wants a line break after the /*
https://review.coreboot.org/c/coreboot/+/47164/2/src/southbridge/intel/commo... PS2, Line 13: I guess this is what makes lint unhappy
https://review.coreboot.org/c/coreboot/+/47164/2/src/southbridge/intel/commo... File src/southbridge/intel/common/post.c:
https://review.coreboot.org/c/coreboot/+/47164/2/src/southbridge/intel/commo... PS2, Line 15: RPR Shouldn't this be `~RPR`?