It has solved some problems with some Skylake laptops. So far, not very many Skylake machines from Apple, but that should change soon.
The _DSM can also live in an OEM SSDT. But not all machines will have it (or not have it). Requires a detailed look at all native DSDT+SSDTs.
I did. Hence my comments regarding unneeded properties.
There are some things I would not tend to use, but it is not that messy.
A few obvious mistakes:
- ig-platform-id=0d220003 which should be 0x0d220003 (it still works as intended because 0d220003 is interpreted as zero, which is interpreted as "unspecified", which means the default is applied... and... the default is... wait for it... 0x0d220003).
- port limit patch is being used (which, of course, is a mistake, except for use in port discovery)
- power management is wrong (should use ssdtPRgen.sh not ACPI/SSDT/Generate)
Not sure what SlpSmiAtWake does. Nor why PatchAPIC is needed (or what it does). These would require investigation/research (eg. reading the source code for Clover).