build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35643 )
Change subject: util/intelp2m: Add Intel Pad to Macro utility ......................................................................
Patch Set 31:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35643/31/util/intelp2m/fields/fsp/f... File util/intelp2m/fields/fsp/fsp.go:
https://review.coreboot.org/c/coreboot/+/35643/31/util/intelp2m/fields/fsp/f... PS31, Line 103: } trailing whitespace
https://review.coreboot.org/c/coreboot/+/35643/31/util/intelp2m/fields/raw/r... File util/intelp2m/fields/raw/raw.go:
https://review.coreboot.org/c/coreboot/+/35643/31/util/intelp2m/fields/raw/r... PS31, Line 16: // Do not decode, print as is. trailing whitespace
https://review.coreboot.org/c/coreboot/+/35643/31/util/intelp2m/platforms/sn... File util/intelp2m/platforms/snr/macro.go:
https://review.coreboot.org/c/coreboot/+/35643/31/util/intelp2m/platforms/sn... PS31, Line 24: // RemmapRstSrc - remmap Pad Reset Source Config trailing whitespace