Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47048 )
Change subject: mb/google/volteer: clang-format mainboard.c ......................................................................
Patch Set 3: Code-Review+2
(1 comment)
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)
ITbtPcieRootPortEn is an array of UINT8, so yes, while the existing code is correctly multiplying th […]
Ah, forgot to mention why I marked this as resolved: this can be fixed in a follow-up commit. I fully agree with your reasoning about not polluting this change, so feel free to simplify this in a follow-up.