From: Tobias Jakobi <[email protected]>
Enables reading the speed of the CPU and GPU fan on the G5.
Signed-off-by: Tobias Jakobi <[email protected]>
---
drivers/hwmon/dell-smm-hwmon.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c
index 44aaf9b9191d..bee9cca2f768 100644
--- a/drivers/hwmon/dell-smm-hwmon.c
+++ b/drivers/hwmon/dell-smm-hwmon.c
@@ -1192,6 +1192,14 @@ static const struct dmi_system_id i8k_dmi_table[] __initconst = {
DMI_MATCH(DMI_PRODUCT_NAME, "XPS"),
},
},
+ {
+ .ident = "Dell G5 5505",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+ DMI_MATCH(DMI_PRODUCT_FAMILY, "GSeries"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "G5 5505"),
+ },
+ },
{ }
};
--
2.43.0
Am 10.03.24 um 18:41 schrieb [email protected]:
> From: Tobias Jakobi <[email protected]>
>
> Enables reading the speed of the CPU and GPU fan on the G5.
>
> Signed-off-by: Tobias Jakobi <[email protected]>
> ---
> drivers/hwmon/dell-smm-hwmon.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c
> index 44aaf9b9191d..bee9cca2f768 100644
> --- a/drivers/hwmon/dell-smm-hwmon.c
> +++ b/drivers/hwmon/dell-smm-hwmon.c
> @@ -1192,6 +1192,14 @@ static const struct dmi_system_id i8k_dmi_table[] __initconst = {
> DMI_MATCH(DMI_PRODUCT_NAME, "XPS"),
> },
> },
> + {
> + .ident = "Dell G5 5505",
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_FAMILY, "GSeries"),
Hi,
i think you can omit the match of DMI_PRODUCT_FAMILY. Also we already have another G5-series machine
in the DMI list (somewhere at the top). Maybe you can put this entry close to this entry?
Thanks,
Armin Wolf
> + DMI_MATCH(DMI_PRODUCT_NAME, "G5 5505"),
> + },
> + },
> { }
> };
>
On 3/10/24 19:26, Armin Wolf wrote:
> Hi,
>
> i think you can omit the match of DMI_PRODUCT_FAMILY. Also we already
> have another G5-series machine
> in the DMI list (somewhere at the top). Maybe you can put this entry
> close to this entry?
Hey,
thanks for the feedback. I've just a sent a v2.
With best wishes,
Tobias
>
> Thanks,
> Armin Wolf
>
>> + DMI_MATCH(DMI_PRODUCT_NAME, "G5 5505"),
>> + },
>> + },
>> { }
>> };
>>