[M] Change in coreboot[master]: common/acpi/dptc: Implement DTTS Proposal

[Tim Van Patten (Code Review)]

Attention is currently required from: Raul Rangel, Jason Nien, EricKY Cheng, Matt DeVillier, Paul Menzel, Martin Roth, Eric Peers, LeilaCY Lin, Jason Glenesk, Caveh Jalali, Tim Wawrzynczak, Fred Reitberger, Karthikeyan Ramasubramanian, Boris Mittelberg, Felix Held.

Tim Van Patten has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68471 )

Change subject: common/acpi/dptc: Implement DTTS Proposal
......................................................................

Patch Set 39:

(5 comments)

File src/soc/amd/common/acpi/DTTS.asl:

https://review.coreboot.org/c/coreboot/+/68471/comment/16f62ffe_8fa85dd2 
PS39, Line 12:     Name(PRTN,7)
Please add a comment indicating the mapping of `0` -> `A`, `1` -> `B`, etc.

https://review.coreboot.org/c/coreboot/+/68471/comment/34d696ce_b8f78b96 
PS39, Line 15:         /* If _SB.DDEF is not present, DPTC is not enabled so return early. */
             :         If (!CondRefOf (_SB.DDEF))
             :         {
             :             Return (Zero)
             :         }
This can be removed once `DTTS()` is called as part of `DPTC()`, which already does this check.

https://review.coreboot.org/c/coreboot/+/68471/comment/7ebe07cd_1dc85405 
PS39, Line 21:         //Set table A as default table after power on SUT
             :         If (_SB.PRTN == 7)
             :         {
             :             _SB.DDEF()
             :             Store(0,_SB.PRTN)
             :             Return (Zero)
             :         }
Can `PRTN` be initialized to `0` so this can be removed?

https://review.coreboot.org/c/coreboot/+/68471/comment/21a03e6f_8c3bd2ed 
PS39, Line 38: 1,\
nit: Various whitespace issues like this, where a space is missing after the `,`.

File src/soc/amd/mendocino/acpi/soc.asl:

https://review.coreboot.org/c/coreboot/+/68471/comment/bb08488d_8e948158 
PS39, Line 33: #if CONFIG(FEATURE_DYNAMIC_DPTC)
             : #include <soc/amd/common/acpi/DTTS.asl>
             : #else
             : #include <soc/amd/common/acpi/dptc.asl>
             : #endif
This removes support for the low/no battery boot feature.   That feature needs to remain supported on all Skyrim devices.

Instead, `dptc.asl` should call `DTTS()` if the device is not in low/no battery mode.