build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26888 )
Change subject: [WIP]sb/intel/common/spi: Add method to probe protected regions ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/26888/1/src/southbridge/intel/common/spi.c File src/southbridge/intel/common/spi.c:
https://review.coreboot.org/#/c/26888/1/src/southbridge/intel/common/spi.c@1... PS1, Line 1026: if (IS_ENABLED(CONFIG_SOUTHBRIDGE_INTEL_I82801GX)) { braces {} are not necessary for any arm of this statement
https://review.coreboot.org/#/c/26888/1/src/southbridge/intel/common/spi.c@1... PS1, Line 1068: if (range_entry_base(entry) <= start && that open brace { should be on the previous line