build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41555 )
Change subject: mb/google/hatch/vr/puff: Setup PL2 and PsysPL2 ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41555/1/src/mainboard/google/hatch/... File src/mainboard/google/hatch/variants/puff/mainboard.c:
https://review.coreboot.org/c/coreboot/+/41555/1/src/mainboard/google/hatch/... PS1, Line 126: /* No data about an arbitary Type-C adapter, set pl4 conservatively. */ 'arbitary' may be misspelled - perhaps 'arbitrary'?
https://review.coreboot.org/c/coreboot/+/41555/1/src/mainboard/google/hatch/... PS1, Line 136: if (igd_id != PCI_DEVICE_ID_INTEL_CML_GT1_ULT_1 && suspect code indent for conditional statements (16, 32)