Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41758 )
Change subject: mb/intel/jasperlake_rvp: Camera remove DSDT and enable SSDT ......................................................................
Patch Set 13:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41758/13/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/41758/13/src/mainboard/intel/jasper... PS13, Line 374: ^
Using ^ in pr0 might cause Windows to BSOD during boot, e.g.:
That's unfortunate, ^ is a perfectly legal part of a namestring, and that would violate the spec.
from the spec: "A name proceeded with ‘^’ causes the name to refer to the parent of the current namespace (‘^’ is not part of the 32-bit fixed-length name)"
You could use the absolute path as well.
I'm not sure if this problem still exists -- at some point coreboot may have changed the > ASL(?) compiler flag so name references are 'absolute' rather than 'relative'.
The acpigen interface emits raw AML bytecode into the SSDT, so this code is not compiled by IASL. And IASL's behavior hasn't changed either, it still emits '^' all over the place, so if that's the case, there may be other entities in an SSDT that could cause the same problem with Windows.