Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44192 )
Change subject: mb/google/zork: Switch USI_RESET to active low polarity for v3.6+ ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44192/2/src/mainboard/google/zork/v... File src/mainboard/google/zork/variants/baseboard/ramstage_common.c:
https://review.coreboot.org/c/coreboot/+/44192/2/src/mainboard/google/zork/v... PS2, Line 213: if (variant_uses_v3_6_schematics()) You should probably add a comment that the rest of the function is for pre v3.6 -- basically what you have in commit description (or a little shorter).
https://review.coreboot.org/c/coreboot/+/44192/2/src/mainboard/google/zork/v... PS2, Line 224: mmio_dev mmio_dev can be NULL here.