Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38743 )
Change subject: nb/intel/haswell/peg: Add PEG driver stub ......................................................................
Patch Set 5:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38743/5/src/northbridge/intel/haswe... File src/northbridge/intel/haswell/acpi/peg.asl:
https://review.coreboot.org/c/coreboot/+/38743/5/src/northbridge/intel/haswe... PS5, Line 17: PEGP
What's "haswell ACPI"? The spec doesn't enforce names at all.
Reference code and the ACPI tables of my main laptop, which is Optimus-capable.
The spec might not enforce any names, but the Nvidia drivers might...
https://review.coreboot.org/c/coreboot/+/38743/5/src/northbridge/intel/haswe... PS5, Line 26: DEV0
No.
Yes. We can be like this all day if you want.
https://review.coreboot.org/c/coreboot/+/38743/5/src/northbridge/intel/haswe... PS5, Line 29: }
This is sometimes used for the dGPU audio device. Out of scope.
Would be nice to know if its lack is what makes the Windows drivers crash.