build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35163 )
Change subject: mb/supermicro: Add X10SLM+-LN4F as X10SLM variant ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... File src/mainboard/supermicro/x10slm/gpio.c:
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... PS1, Line 192: { that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... PS1, Line 201: { that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... PS1, Line 208: { that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... File src/mainboard/supermicro/x10slm/romstage.c:
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... PS1, Line 64: { that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/35163/1/src/mainboard/supermicro/x1... PS1, Line 82: { that open brace { should be on the previous line