* [PATCH] hwmon: (pmbus/isl68137) remove READ_TEMPERATURE_3 for RAA228228
@ 2021-05-14 21:19 Grant Peltier
2021-05-14 21:43 ` Guenter Roeck
0 siblings, 1 reply; 3+ messages in thread
From: Grant Peltier @ 2021-05-14 21:19 UTC (permalink / raw)
To: linux; +Cc: linux-hwmon, grant.peltier.jg, adam.vaughn.xh
The initial version of the RAA228228 datasheet claimed that the device
supported READ_TEMPERATURE_3 but not READ_TEMPERATURE_1. It has since been
discovered that the datasheet was incorrect. The RAA228228 does support
READ_TEMPERATURE_1 but does not support READ_TEMPERATURE_3.
This change fixes an error introduced in commit:
51fb91ed5a6fa855a74731610cd5435d83d6e17f
Signed-off-by: Grant Peltier <grantpeltier93@gmail.com>
---
drivers/hwmon/pmbus/isl68137.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/hwmon/pmbus/isl68137.c b/drivers/hwmon/pmbus/isl68137.c
index 40597a9e799f..1a8caff1ac5f 100644
--- a/drivers/hwmon/pmbus/isl68137.c
+++ b/drivers/hwmon/pmbus/isl68137.c
@@ -244,8 +244,8 @@ static int isl68137_probe(struct i2c_client *client)
info->read_word_data = raa_dmpvr2_read_word_data;
break;
case raa_dmpvr2_2rail_nontc:
- info->func[0] &= ~PMBUS_HAVE_TEMP;
- info->func[1] &= ~PMBUS_HAVE_TEMP;
+ info->func[0] &= ~PMBUS_HAVE_TEMP3;
+ info->func[1] &= ~PMBUS_HAVE_TEMP3;
fallthrough;
case raa_dmpvr2_2rail:
info->pages = 2;
--
2.20.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] hwmon: (pmbus/isl68137) remove READ_TEMPERATURE_3 for RAA228228
2021-05-14 21:19 [PATCH] hwmon: (pmbus/isl68137) remove READ_TEMPERATURE_3 for RAA228228 Grant Peltier
@ 2021-05-14 21:43 ` Guenter Roeck
[not found] ` <OS3PR01MB64529FC38D0AD138EC814DCCD2509@OS3PR01MB6452.jpnprd01.prod.outlook.com>
0 siblings, 1 reply; 3+ messages in thread
From: Guenter Roeck @ 2021-05-14 21:43 UTC (permalink / raw)
To: Grant Peltier; +Cc: linux-hwmon, grant.peltier.jg, adam.vaughn.xh
On 5/14/21 2:19 PM, Grant Peltier wrote:
> The initial version of the RAA228228 datasheet claimed that the device
> supported READ_TEMPERATURE_3 but not READ_TEMPERATURE_1. It has since been
> discovered that the datasheet was incorrect. The RAA228228 does support
> READ_TEMPERATURE_1 but does not support READ_TEMPERATURE_3.
>
Sorry that I have to ask, but do you have some kind of reference for that ?
Due to recent events I can unfortunately no longer trust such submissions
without some kind of confirmation (for example resend from your Renesas
e-mail address if that is possible, or ask someone from a Renesas e-mail
address to send a Reviewed-by: / Acked-by:).
> This change fixes an error introduced in commit:
> 51fb91ed5a6fa855a74731610cd5435d83d6e17f
>
That should be
Fixes: 51fb91ed5a6fa ("hwmon: (pmbus/isl68137) remove READ_TEMPERATURE_1 telemetry for RAA228228")
Thanks,
Guenter
> Signed-off-by: Grant Peltier <grantpeltier93@gmail.com>
> ---
> drivers/hwmon/pmbus/isl68137.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/hwmon/pmbus/isl68137.c b/drivers/hwmon/pmbus/isl68137.c
> index 40597a9e799f..1a8caff1ac5f 100644
> --- a/drivers/hwmon/pmbus/isl68137.c
> +++ b/drivers/hwmon/pmbus/isl68137.c
> @@ -244,8 +244,8 @@ static int isl68137_probe(struct i2c_client *client)
> info->read_word_data = raa_dmpvr2_read_word_data;
> break;
> case raa_dmpvr2_2rail_nontc:
> - info->func[0] &= ~PMBUS_HAVE_TEMP;
> - info->func[1] &= ~PMBUS_HAVE_TEMP;
> + info->func[0] &= ~PMBUS_HAVE_TEMP3;
> + info->func[1] &= ~PMBUS_HAVE_TEMP3;
> fallthrough;
> case raa_dmpvr2_2rail:
> info->pages = 2;
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-05-14 22:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-14 21:19 [PATCH] hwmon: (pmbus/isl68137) remove READ_TEMPERATURE_3 for RAA228228 Grant Peltier
2021-05-14 21:43 ` Guenter Roeck
[not found] ` <OS3PR01MB64529FC38D0AD138EC814DCCD2509@OS3PR01MB6452.jpnprd01.prod.outlook.com>
2021-05-14 22:23 ` 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).