[PATCH 2/2] platform/x86: acer-wmi: enable hwmon on Aspire A315-58G

Mohsen Tahmasebi <[email protected]>
Newsgroups org.kernel.vger.platform-driver-x86,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
The Acer Aspire A315-58G supports the Acer hwmon interface and reports
CPU, GPU and external temperatures.
Note that the GPU temperature reports 0 when the Nvidia MX350 GPU is in
D3cold, but reports a valid value when the GPU is in D3hot/D0.

This commit enables hwmon for this model.

Signed-off-by: Mohsen Tahmasebi <[email protected]>
---

Tested on Acer Aspire A315-58G (BIOS V1.35).

With this series applied and MX350 GPU in D3hot:

  $ cat /sys/class/dmi/id/sys_vendor
  Acer

  $ cat /sys/class/dmi/id/product_name
  Aspire A315-58G

  $ sensors
  ...
  acer-isa-0ace
  Adapter: ISA adapter
  temp1:        +57.0°C
  temp2:        +50.0°C
  temp3:        +56.0°C
  ...

Sensor mapping:
  temp1: CPU temperature
  temp2: GPU temperature
  temp3: external temperature


 drivers/platform/x86/acer-wmi.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
index 83448561189..d251752d51e 100644
--- a/drivers/platform/x86/acer-wmi.c
+++ b/drivers/platform/x86/acer-wmi.c
@@ -455,6 +455,10 @@ static struct quirk_entry quirk_acer_aspire_1520 = {
 	.brightness = -1,
 };
 
+static struct quirk_entry quirk_acer_aspire_a315_58g = {
+	.hwmon = 1,
+};
+
 static struct quirk_entry quirk_acer_travelmate_2490 = {
 	.mailled = 1,
 };
@@ -639,6 +643,15 @@ static const struct dmi_system_id acer_quirks[] __initconst = {
 		},
 		.driver_data = &quirk_acer_travelmate_2490,
 	},
+	{
+		.callback = dmi_matched,
+		.ident = "Acer Aspire A315-58G",
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire A315-58G"),
+		},
+		.driver_data = &quirk_acer_aspire_a315_58g,
+	},
 	{
 		.callback = dmi_matched,
 		.ident = "Acer TravelMate 2490",
-- 
2.54.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.