build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49296 )
Change subject: mb/google/zork/var/vilboz: SAR config ......................................................................
Patch Set 1:
(4 comments)
File src/mainboard/google/zork/variants/vilboz/variant.c:
https://review.coreboot.org/c/coreboot/+/49296/comment/65e33599_884a2862 PS1, Line 67: switch (sar_config) that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/49296/comment/5884dc82_e541dc36 PS1, Line 73: //TODO: Set default first. It will be replaced after the new table is generated. line over 96 characters
https://review.coreboot.org/c/coreboot/+/49296/comment/b148cda3_f5d7c50a PS1, Line 80: //TODO: Set default first. It will be replaced after the new table is generated. line over 96 characters
https://review.coreboot.org/c/coreboot/+/49296/comment/940f70c3_133672b3 PS1, Line 87: return filename; please, no spaces at the start of a line