Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38479 )
Change subject: mb/google/puff: Add puff ac/dc loadline configurations ......................................................................
Patch Set 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/38479/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38479/2//COMMIT_MSG@7 PS2, Line 7: puff Already in the prefix.
https://review.coreboot.org/c/coreboot/+/38479/2//COMMIT_MSG@9 PS2, Line 9: VRTT What does this stand for?
https://review.coreboot.org/c/coreboot/+/38479/2//COMMIT_MSG@10 PS2, Line 10: in puff device tree. Some words fit on line above (75 characters).
https://review.coreboot.org/c/coreboot/+/38479/2//COMMIT_MSG@11 PS2, Line 11: Did it boot up before also?