4 comments:

File src/soc/intel/tigerlake/acpi/tcss.asl:
- Patch Set #7, Line 329: Printf("Bring TBT RPs out of D3Code.")
  these seem like they could still be useful?
- Patch Set #7, Line 145: {
  pleas pull these { to the same line as If() statements for consistency
- Patch Set #7, Line 162: prior booting to OS.
  this comment about 'prior to booting to OS' seems out of place in a runtime hotplug event.
- Patch Set #7, Line 257:
```
		\_SB.PCI0.TRP0.HPME()
		\_SB.PCI0.TRP1.HPME()
		\_SB.PCI0.TRP2.HPME()
		\_SB.PCI0.TRP3.HPME()
```
  should this be in Else() so it doesn't consume a PME notification? or if this is intentional can you add a comment.

To view, visit change 42149. To unsubscribe, or for help writing mail filters, visit settings.