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

[EricKY Cheng (Code Review)]

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

EricKY Cheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68471 )

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

Patch Set 41:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/68471/comment/ed48afb5_246af2c0 
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.

Based on your suggestion, I need your help to clarify the design thinking as below.
You would like to request the code change as below?

1. Include DTTS.asl in DTPC.asl and separate by FEATURE_DYNAMIC_DPTC.

method(DPTC)
{
  /* If _SB.DDEF is not present, DPTC is not enabled so return early. */
  If (!CondRefOf (_SB.DDEF))
  {
    Return (Zero)
  }

No/Low battery ...etc

#if CONFIG(FEATURE_DYNAMIC_DPTC)

#include <soc/amd/common/acpi/DTTS.asl>

#endif
}

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

https://review.coreboot.org/c/coreboot/+/68471/comment/bcfe9507_a9a8dc4b 
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. […]

Follow thermal team's request, DTTS() is designed as customize feature.
Thermal team didn't ask low/no battery mode on the proposal.
DPTC() and DTTS() are independent thermal control methods separate by FEATURE_DYNAMIC_DPTC.

Based on your suggestion, I need your help to clarify the design thinking as below.
You would like to request the code change as below?

1. Include DTTS.asl in DTPC.asl and separate by FEATURE_DYNAMIC_DPTC.

method(DPTC)
{
  /* If _SB.DDEF is not present, DPTC is not enabled so return early. */
  If (!CondRefOf (_SB.DDEF))
  {
    Return (Zero)
  }

No/Low battery ...etc

#if CONFIG(FEATURE_DYNAMIC_DPTC)

#include <soc/amd/common/acpi/DTTS.asl>

#endif
}