Matthew Ziegelbaum has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47556 )
Change subject: mb/google/hatch/var/ambassador: update fan table and tdp config ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47556/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/47556/2//COMMIT_MSG@9 PS2, Line 9: Fan table: provided by the ODM (see attachment in bug) based on : measurements with EVT unit. : : USB config: based on USB 3.1 gen 2 tuning done for Puff : (see http://b/150515720).
Two items warrant two separate commits.
Done--after talking with the ODM I'm going to hold off on a USB change, but I split out the FSP option change to its own CL in https://review.coreboot.org/c/coreboot/+/47685
https://review.coreboot.org/c/coreboot/+/47556/2//COMMIT_MSG@13 PS2, Line 13: http://
Remove, as it’s not a valid URL?
Done