Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47048 )
Change subject: mb/google/volteer: clang-format of mainboard.c ......................................................................
Patch Set 2: Code-Review+2
(3 comments)
https://review.coreboot.org/c/coreboot/+/47048/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/47048/2//COMMIT_MSG@7 PS2, Line 7: of nit: drop `of`
https://review.coreboot.org/c/coreboot/+/47048/2//COMMIT_MSG@9 PS2, Line 9: autmatic nit: aut*o*matic
https://review.coreboot.org/c/coreboot/+/47048/2/src/mainboard/google/voltee... File src/mainboard/google/volteer/mainboard.c:
https://review.coreboot.org/c/coreboot/+/47048/2/src/mainboard/google/voltee... PS2, Line 141: ARRAY_SIZE(params->ITbtPcieRootPortEn) : * sizeof(*params->ITbtPcieRootPortEn) shouldn't this just be `sizeof(params->ITbtPcieRootPortEn)`?