* [PATCH] hwmon: (pmbus/max16601) Add MAX16602 support
@ 2022-04-15 14:06 Ofluoglu, Atif
2022-04-24 16:19 ` Guenter Roeck
0 siblings, 1 reply; 2+ messages in thread
From: Ofluoglu, Atif @ 2022-04-15 14:06 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Ofluoglu, Atif, linux-hwmon, linux-kernel, Jean Delvare
From e2a1f7d1a836188a9b284fc473af9118aa160ddb Mon Sep 17 00:00:00 2001
From: Atif Ofluoglu <mailto:atif.ofluoglu@maximintegrated.com>
Date: Fri, 15 Apr 2022 16:34:29 +0300
Subject: [PATCH] hwmon: (pmbus/max16601) Add MAX16602 support
Adding another MAX16602 chip support to MAX16601 driver
Tested with MAX16602 works as expected.
Signed-off-by: Atif Ofluoglu <mailto:atif.ofluoglu@maximintegrated.com>
---
Documentation/hwmon/max16601.rst | 8 ++++++++
drivers/hwmon/pmbus/Kconfig | 4 ++--
drivers/hwmon/pmbus/max16601.c | 13 ++++++++-----
3 files changed, 18 insertions(+), 7 deletions(-)
diff --git a/Documentation/hwmon/max16601.rst b/Documentation/hwmon/max16601.rst
index 92c0a7d7808c..6a4eef8efbaf 100644
--- a/Documentation/hwmon/max16601.rst
+++ b/Documentation/hwmon/max16601.rst
@@ -21,6 +21,14 @@ Supported chips:
Datasheet: Not published
+ * Maxim MAX16602
+
+ Prefix: 'max16602'
+
+ Addresses scanned: -
+
+ Datasheet: https://datasheets.maximintegrated.com/en/ds/MAX16602.pdf
+
Author: Guenter Roeck <mailto:linux@roeck-us.net>
diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
index a2ea1d5a8765..53683d2d0fb2 100644
--- a/drivers/hwmon/pmbus/Kconfig
+++ b/drivers/hwmon/pmbus/Kconfig
@@ -228,10 +228,10 @@ config SENSORS_MAX16064
be called max16064.
config SENSORS_MAX16601
- tristate "Maxim MAX16508, MAX16601"
+ tristate "Maxim MAX16508, MAX16601, MAX16602"
help
If you say yes here you get hardware monitoring support for Maxim
- MAX16508 and MAX16601.
+ MAX16508, MAX16601 and MAX16602.
This driver can also be built as a module. If so, the module will
be called max16601.
diff --git a/drivers/hwmon/pmbus/max16601.c b/drivers/hwmon/pmbus/max16601.c
index 5a226a564776..b628405e6586 100644
--- a/drivers/hwmon/pmbus/max16601.c
+++ b/drivers/hwmon/pmbus/max16601.c
@@ -1,6 +1,6 @@
// SPDX-License-Identifier: GPL-2.0
/*
- * Hardware monitoring driver for Maxim MAX16508 and MAX16601.
+ * Hardware monitoring driver for Maxim MAX16508, MAX16601 and MAX16602.
*
* Implementation notes:
*
@@ -31,7 +31,7 @@
#include "pmbus.h"
-enum chips { max16508, max16601 };
+enum chips { max16508, max16601, max16602 };
#define REG_DEFAULT_NUM_POP 0xc4
#define REG_SETPT_DVID 0xd1
@@ -202,7 +202,7 @@ static int max16601_identify(struct i2c_client *client,
else
info->vrm_version[0] = vr12;
- if (data->id != max16601)
+ if (data->id != max16601 && data->id != max16602)
return 0;
reg = i2c_smbus_read_byte_data(client, REG_DEFAULT_NUM_POP);
@@ -264,6 +264,7 @@ static void max16601_remove(void *_data)
static const struct i2c_device_id max16601_id[] = {
{"max16508", max16508},
{"max16601", max16601},
+ {"max16602", max16602},
{}
};
MODULE_DEVICE_TABLE(i2c, max16601_id);
@@ -280,13 +281,15 @@ static int max16601_get_id(struct i2c_client *client)
return -ENODEV;
/*
- * PMBUS_IC_DEVICE_ID is expected to return "MAX16601y.xx"
- * or "MAX16500y.xx".
+ * PMBUS_IC_DEVICE_ID is expected to return "MAX16601y.xx" or
+ * MAX16602y.xx or "MAX16500y.xx".cdxxcccccccccc
*/
if (!strncmp(buf, "MAX16500", 8)) {
id = max16508;
} else if (!strncmp(buf, "MAX16601", 8)) {
id = max16601;
+ } else if (!strncmp(buf, "MAX16602", 8)) {
+ id = max16602;
} else {
buf[ret] = '\0';
dev_err(dev, "Unsupported chip '%s'\n", buf);
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] hwmon: (pmbus/max16601) Add MAX16602 support
2022-04-15 14:06 [PATCH] hwmon: (pmbus/max16601) Add MAX16602 support Ofluoglu, Atif
@ 2022-04-24 16:19 ` Guenter Roeck
0 siblings, 0 replies; 2+ messages in thread
From: Guenter Roeck @ 2022-04-24 16:19 UTC (permalink / raw)
To: Ofluoglu, Atif; +Cc: linux-hwmon, linux-kernel, Jean Delvare
On 4/15/22 07:06, Ofluoglu, Atif wrote:
>>From e2a1f7d1a836188a9b284fc473af9118aa160ddb Mon Sep 17 00:00:00 2001
> From: Atif Ofluoglu <mailto:atif.ofluoglu@maximintegrated.com>
> Date: Fri, 15 Apr 2022 16:34:29 +0300
> Subject: [PATCH] hwmon: (pmbus/max16601) Add MAX16602 support
>
> Adding another MAX16602 chip support to MAX16601 driver
> Tested with MAX16602 works as expected.
>
> Signed-off-by: Atif Ofluoglu <mailto:atif.ofluoglu@maximintegrated.com>
Never mind, applied. The problem was that your e-mail system adds
"mailto:" to all e-mail addresses. See above and in the patch.
Also, the actual patch was embedded in another e-mail frame which
confused git. After fixing that up, that the patch applied cleanly.
In the future, please make sure that your e-mail system leaves
any patches you send alone.
Thanks,
Guenter
> ---
> Documentation/hwmon/max16601.rst | 8 ++++++++
> drivers/hwmon/pmbus/Kconfig | 4 ++--
> drivers/hwmon/pmbus/max16601.c | 13 ++++++++-----
> 3 files changed, 18 insertions(+), 7 deletions(-)
>
> diff --git a/Documentation/hwmon/max16601.rst b/Documentation/hwmon/max16601.rst
> index 92c0a7d7808c..6a4eef8efbaf 100644
> --- a/Documentation/hwmon/max16601.rst
> +++ b/Documentation/hwmon/max16601.rst
> @@ -21,6 +21,14 @@ Supported chips:
>
> Datasheet: Not published
>
> + * Maxim MAX16602
> +
> + Prefix: 'max16602'
> +
> + Addresses scanned: -
> +
> + Datasheet: https://datasheets.maximintegrated.com/en/ds/MAX16602.pdf
> +
> Author: Guenter Roeck <mailto:linux@roeck-us.net>
>
>
> diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
> index a2ea1d5a8765..53683d2d0fb2 100644
> --- a/drivers/hwmon/pmbus/Kconfig
> +++ b/drivers/hwmon/pmbus/Kconfig
> @@ -228,10 +228,10 @@ config SENSORS_MAX16064
> be called max16064.
>
> config SENSORS_MAX16601
> - tristate "Maxim MAX16508, MAX16601"
> + tristate "Maxim MAX16508, MAX16601, MAX16602"
> help
> If you say yes here you get hardware monitoring support for Maxim
> - MAX16508 and MAX16601.
> + MAX16508, MAX16601 and MAX16602.
>
> This driver can also be built as a module. If so, the module will
> be called max16601.
> diff --git a/drivers/hwmon/pmbus/max16601.c b/drivers/hwmon/pmbus/max16601.c
> index 5a226a564776..b628405e6586 100644
> --- a/drivers/hwmon/pmbus/max16601.c
> +++ b/drivers/hwmon/pmbus/max16601.c
> @@ -1,6 +1,6 @@
> // SPDX-License-Identifier: GPL-2.0
> /*
> - * Hardware monitoring driver for Maxim MAX16508 and MAX16601.
> + * Hardware monitoring driver for Maxim MAX16508, MAX16601 and MAX16602.
> *
> * Implementation notes:
> *
> @@ -31,7 +31,7 @@
>
> #include "pmbus.h"
>
> -enum chips { max16508, max16601 };
> +enum chips { max16508, max16601, max16602 };
>
> #define REG_DEFAULT_NUM_POP 0xc4
> #define REG_SETPT_DVID 0xd1
> @@ -202,7 +202,7 @@ static int max16601_identify(struct i2c_client *client,
> else
> info->vrm_version[0] = vr12;
>
> - if (data->id != max16601)
> + if (data->id != max16601 && data->id != max16602)
> return 0;
>
> reg = i2c_smbus_read_byte_data(client, REG_DEFAULT_NUM_POP);
> @@ -264,6 +264,7 @@ static void max16601_remove(void *_data)
> static const struct i2c_device_id max16601_id[] = {
> {"max16508", max16508},
> {"max16601", max16601},
> + {"max16602", max16602},
> {}
> };
> MODULE_DEVICE_TABLE(i2c, max16601_id);
> @@ -280,13 +281,15 @@ static int max16601_get_id(struct i2c_client *client)
> return -ENODEV;
>
> /*
> - * PMBUS_IC_DEVICE_ID is expected to return "MAX16601y.xx"
> - * or "MAX16500y.xx".
> + * PMBUS_IC_DEVICE_ID is expected to return "MAX16601y.xx" or
> + * MAX16602y.xx or "MAX16500y.xx".cdxxcccccccccc
> */
> if (!strncmp(buf, "MAX16500", 8)) {
> id = max16508;
> } else if (!strncmp(buf, "MAX16601", 8)) {
> id = max16601;
> + } else if (!strncmp(buf, "MAX16602", 8)) {
> + id = max16602;
> } else {
> buf[ret] = '\0';
> dev_err(dev, "Unsupported chip '%s'\n", buf);
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-04-24 16:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-15 14:06 [PATCH] hwmon: (pmbus/max16601) Add MAX16602 support Ofluoglu, Atif
2022-04-24 16:19 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).