Subject: [PATCH] hwmon: (oxp-sensors) Add support for AOKZOE A1 PRO

From: Jerrod Frost <[email protected]>

This device is an iteration over the AOKZOE A1 with the same EC mapping
and features. It also has support for tt_toggle.

Thanks to Jerrod for authoring and testing.

Signed-off-by: Joaquín Ignacio Aramendía <[email protected]>
---
Documentation/hwmon/oxp-sensors.rst | 2 ++
drivers/hwmon/oxp-sensors.c | 7 +++++++
2 files changed, 9 insertions(+)

diff --git a/Documentation/hwmon/oxp-sensors.rst b/Documentation/hwmon/oxp-sensors.rst
index 0ca1f7728c34..3adeb7406243 100644
--- a/Documentation/hwmon/oxp-sensors.rst
+++ b/Documentation/hwmon/oxp-sensors.rst
@@ -30,6 +30,7 @@ Supported devices
Currently the driver supports the following handhelds:

- AOK ZOE A1
+ - AOK ZOE A1 PRO
- Aya Neo 2
- Aya Neo AIR
- Aya Neo AIR Pro
@@ -40,6 +41,7 @@ Currently the driver supports the following handhelds:

"Turbo/Silent" button behaviour toggle is only supported on:
- AOK ZOE A1
+ - AOK ZOE A1 PRO
- OneXPlayer mini AMD (only with updated alpha BIOS)
- OneXPlayer mini AMD PRO

diff --git a/drivers/hwmon/oxp-sensors.c b/drivers/hwmon/oxp-sensors.c
index 1e59d97219c4..e1a907cae820 100644
--- a/drivers/hwmon/oxp-sensors.c
+++ b/drivers/hwmon/oxp-sensors.c
@@ -77,6 +77,13 @@ static const struct dmi_system_id dmi_table[] = {
},
.driver_data = (void *)aok_zoe_a1,
},
+ {
+ .matches = {
+ DMI_MATCH(DMI_BOARD_VENDOR, "AOKZOE"),
+ DMI_EXACT_MATCH(DMI_BOARD_NAME, "AOKZOE A1 Pro"),
+ },
+ .driver_data = (void *)aok_zoe_a1,
+ },
{
.matches = {
DMI_MATCH(DMI_BOARD_VENDOR, "AYANEO"),
--
2.41.0



2023-06-25 00:33:14

by Guenter Roeck

[permalink] [raw]
Subject: Re: [PATCH] hwmon: (oxp-sensors) Add support for AOKZOE A1 PRO

On 6/24/23 16:16, Joaquín Ignacio Aramendía wrote:
> From: Jerrod Frost <[email protected]>
>
> This device is an iteration over the AOKZOE A1 with the same EC mapping
> and features. It also has support for tt_toggle.
>
> Thanks to Jerrod for authoring and testing.
>
> Signed-off-by: Joaquín Ignacio Aramendía <[email protected]>

From: and the first Signed-off-by: have to match. You would need
Jerrod's Signed-off-by: tag to keep his authorship.

Guenter

> ---
> Documentation/hwmon/oxp-sensors.rst | 2 ++
> drivers/hwmon/oxp-sensors.c | 7 +++++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/Documentation/hwmon/oxp-sensors.rst b/Documentation/hwmon/oxp-sensors.rst
> index 0ca1f7728c34..3adeb7406243 100644
> --- a/Documentation/hwmon/oxp-sensors.rst
> +++ b/Documentation/hwmon/oxp-sensors.rst
> @@ -30,6 +30,7 @@ Supported devices
> Currently the driver supports the following handhelds:
>
> - AOK ZOE A1
> + - AOK ZOE A1 PRO
> - Aya Neo 2
> - Aya Neo AIR
> - Aya Neo AIR Pro
> @@ -40,6 +41,7 @@ Currently the driver supports the following handhelds:
>
> "Turbo/Silent" button behaviour toggle is only supported on:
> - AOK ZOE A1
> + - AOK ZOE A1 PRO
> - OneXPlayer mini AMD (only with updated alpha BIOS)
> - OneXPlayer mini AMD PRO
>
> diff --git a/drivers/hwmon/oxp-sensors.c b/drivers/hwmon/oxp-sensors.c
> index 1e59d97219c4..e1a907cae820 100644
> --- a/drivers/hwmon/oxp-sensors.c
> +++ b/drivers/hwmon/oxp-sensors.c
> @@ -77,6 +77,13 @@ static const struct dmi_system_id dmi_table[] = {
> },
> .driver_data = (void *)aok_zoe_a1,
> },
> + {
> + .matches = {
> + DMI_MATCH(DMI_BOARD_VENDOR, "AOKZOE"),
> + DMI_EXACT_MATCH(DMI_BOARD_NAME, "AOKZOE A1 Pro"),
> + },
> + .driver_data = (void *)aok_zoe_a1,
> + },
> {
> .matches = {
> DMI_MATCH(DMI_BOARD_VENDOR, "AYANEO"),