Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42455 )
Change subject: nb/intel/sandybridge/gma.c: Correct deasserting force wake on IVB ......................................................................
Patch Set 2:
(2 comments)
I know it's a lot of work, but I think all the changes in this series (even individual bits) should be tested separately if they fix anything.
https://review.coreboot.org/c/coreboot/+/42455/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42455/2//COMMIT_MSG@7 PS2, Line 7: nb/intel/sandybridge/gma.c: Correct deasserting force wake on IVB How can we tell if this is indeed correcting something?
https://review.coreboot.org/c/coreboot/+/42455/2//COMMIT_MSG@9 PS2, Line 9: Reference *Somebody else's*?