Attention is currently required from: Kevin Chang, YH Lin. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59973 )
Change subject: mb/google/brya/var/taeko: Fix PLD group order(W/A) ......................................................................
Patch Set 5:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/59973/comment/c179338f_24e429a4 PS5, Line 7: (W/A) What does W/A emean?
https://review.coreboot.org/c/coreboot/+/59973/comment/7719707e_da109db2 PS5, Line 7: order(W/A) Please add a space before (.
https://review.coreboot.org/c/coreboot/+/59973/comment/f388628a_8dcc2b57 PS5, Line 9: In ec/google/chromeec: Add PLD to EC conn in ACPI table(667471b8d807da5a5a9277db47e069ad3b1351c7) In commit 667471b8d8 (ec/google/chromeec: Add PLD to EC conn in ACPI table) …
https://review.coreboot.org/c/coreboot/+/59973/comment/2f97a77a_aa599444 PS5, Line 9: In ec/google/chromeec: Add PLD to EC conn in ACPI table(667471b8d807da5a5a9277db47e069ad3b1351c7), PLD is added to ACPI table. It causes the DUT to not boot into the OS. So fix the USB3/USB2 Type-C Port C2 PLD group order from 3 to 2 to solve this issue. Please reflow for 75 characters per line.
https://review.coreboot.org/c/coreboot/+/59973/comment/2c7108c1_2e8b9f85 PS5, Line 9: table(667471b8d807da5a5a9277db47e069ad3b1351c7) Please add a space before (.
https://review.coreboot.org/c/coreboot/+/59973/comment/f99ee9d0_4454b0ee PS5, Line 10: Fixes: 667471b8d8 ("ec/google/chromeec: Add PLD to EC conn in ACPI table")
-
Paul Menzel (Code Review)