[coreboot-gerrit] Change in coreboot[master]: drivers/intel/gma: Include gfx.asl by default for all platforms...

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48862 )

Change subject: drivers/intel/gma: Include gfx.asl by default for all platforms...
......................................................................

Patch Set 5: Code-Review+2

(3 comments)

https://review.coreboot.org/c/coreboot/+/48862/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/48862/5//COMMIT_MSG@7 
PS5, Line 7: ...
           : 
           : which
nit: maybe move the ... in front of "which select" - just my opinion :)

https://review.coreboot.org/c/coreboot/+/48862/5//COMMIT_MSG@15 
PS5, Line 15: eg
e.g. Win...

https://review.coreboot.org/c/coreboot/+/48862/5/src/mainboard/51nb/x210/acp... 
File src/mainboard/51nb/x210/acpi/graphics.asl:

https://review.coreboot.org/c/coreboot/+/48862/5/src/mainboard/51nb/x210/acp... 
PS5, Line 5: 
nit: drop that newline?