build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44661 )
Change subject: mb/google/dedede/var/drawcia: Add Wifi SAR for drawcia ......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/44661/7/src/mainboard/google/dedede... File src/mainboard/google/dedede/variants/drawcia/fw_config.c:
https://review.coreboot.org/c/coreboot/+/44661/7/src/mainboard/google/dedede... PS7, Line 12: if (fw_config_probe(FW_CONFIG(TABLETMODE, TABLETMODE_ENABLED))) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/44661/7/src/mainboard/google/dedede... PS7, Line 14: } code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/44661/7/src/mainboard/google/dedede... PS7, Line 14: } please, no spaces at the start of a line