[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Configure TCSS power management

[Duncan Laurie (Code Review)]

Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39785 )

Change subject: soc/intel/tigerlake: Configure TCSS power management
......................................................................

Patch Set 21:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39785/18/src/soc/intel/tigerlake/ac... 
File src/soc/intel/tigerlake/acpi/tcss.asl:

https://review.coreboot.org/c/coreboot/+/39785/18/src/soc/intel/tigerlake/ac... 
PS18, Line 172: * DBG("Timeout occurred.") */
> DBG is the pseudo name. APRT is the actual method for acpi debug.

Printf() will output to the debug object which goes to /dev/null by default.  So if you just leave these in place and uncommented with Printf() they will be silenced by default but will be able to be enabled with a simple:  echo 1 > /sys/module/acpi/parameters/aml_debug_output

I think this will be useful for s0ix testing and debug without needing a special built BIOS.