Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42645 )
Change subject: sb/intel/i82801{gx,jx}/nvs.h: Add include guards ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42645/1/src/southbridge/intel/i8280... File src/southbridge/intel/i82801gx/nvs.h:
https://review.coreboot.org/c/coreboot/+/42645/1/src/southbridge/intel/i8280... PS1, Line 5: #include <stdint.h>
maybe insert an empty line between include and the guard ?
i82801ix does not have any spaces. Moreover, this change is to make diffstats more meaningful, so adding a space in this commit would just add noise.