Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/57881 )
Change subject: ec/system76/ec: acpi: Add dGPU thermal reporting ......................................................................
ec/system76/ec: acpi: Add dGPU thermal reporting
Add a new config for boards with dGPUs to enable reporting fan duty and temperature. The dGPU is not yet enabled on any boards, so it always reports the temp as 0. However, the EC firmware does use the dGPU's fan and so reports valid information for fan speed.
Change-Id: Iae1063ee6a082a77ed026178eb9471bbc2b2fadf Signed-off-by: Jeremy Soller jeremy@system76.com Signed-off-by: Tim Crawford tcrawford@system76.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/57881 Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/ec/system76/ec/Kconfig M src/ec/system76/ec/acpi/s76.asl 2 files changed, 11 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Jeremy Soller: Looks good to me, approved
diff --git a/src/ec/system76/ec/Kconfig b/src/ec/system76/ec/Kconfig index e9b8091..499b7f8 100644 --- a/src/ec/system76/ec/Kconfig +++ b/src/ec/system76/ec/Kconfig @@ -13,6 +13,11 @@ bool default n
+config EC_SYSTEM76_EC_DGPU + depends on EC_SYSTEM76_EC + bool + default n + config EC_SYSTEM76_EC_OLED depends on EC_SYSTEM76_EC bool diff --git a/src/ec/system76/ec/acpi/s76.asl b/src/ec/system76/ec/acpi/s76.asl index 62a93ba..d641ada 100644 --- a/src/ec/system76/ec/acpi/s76.asl +++ b/src/ec/system76/ec/acpi/s76.asl @@ -117,6 +117,9 @@ Method (NFAN, 0, Serialized) { Return (Package() { "CPU fan", +#if CONFIG(EC_SYSTEM76_EC_DGPU) + "GPU fan", +#endif }) }
@@ -144,6 +147,9 @@ Method (NTMP, 0, Serialized) { Return (Package() { "CPU temp", +#if CONFIG(EC_SYSTEM76_EC_DGPU) + "GPU temp", +#endif }) }