build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32280 )
Change subject: mb/google/sarien: Update SMBIOS type17 ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/32280/1/src/mainboard/google/sarien/ramstage... File src/mainboard/google/sarien/ramstage.c:
https://review.coreboot.org/#/c/32280/1/src/mainboard/google/sarien/ramstage... PS1, Line 27: switch(dimm->channel_num) { space required before the open parenthesis '('
https://review.coreboot.org/#/c/32280/1/src/mainboard/google/sarien/ramstage... PS1, Line 38: "UNKOWN"); 'UNKOWN' may be misspelled - perhaps 'UNKNOWN'?