diff mbox series

[2/2] hwmon: (lm75) Add NXP P3T1755 support

Message ID 20241217120304.32950-6-wsa+renesas@sang-engineering.com (mailing list archive)
State New
Delegated to: Geert Uytterhoeven
Headers show
Series hwmon: (lm75) Add NXP P3T1755 support | expand

Commit Message

Wolfram Sang Dec. 17, 2024, 12:03 p.m. UTC 
Tested with the P3T1755 soldered additionally to the P3T1085UK-ARD
shield and connected to a Renesas RZ/G3S SMARC board.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 Documentation/hwmon/lm75.rst |  6 ++++--
 drivers/hwmon/lm75.c         | 13 +++++++++++++
 2 files changed, 17 insertions(+), 2 deletions(-)

Comments

Guenter Roeck Dec. 17, 2024, 9:26 p.m. UTC | #1 
Hi Wolfram,

On 12/17/24 04:03, Wolfram Sang wrote:
> Tested with the P3T1755 soldered additionally to the P3T1085UK-ARD
> shield and connected to a Renesas RZ/G3S SMARC board.
> 

Technically that should describe the patch, not how you tested it.

Anyway, can you send me a register dump for this chip ? I'd like to
add it to my module test scripts.

Thanks,
Guenter
Wolfram Sang Dec. 18, 2024, 7:22 a.m. UTC | #2 
> Technically that should describe the patch, not how you tested it.

I will modify it.

> Anyway, can you send me a register dump for this chip ? I'd like to
> add it to my module test scripts.

Sure thing. What format? Plain hex values in ASCII?
Wolfram Sang Dec. 18, 2024, 9:15 a.m. UTC | #3 
> Sure thing. What format? Plain hex values in ASCII?

Reading registers from the bootloader matches the default values in the
datasheet:

=> i2c md 0x4c 0 2
0000: 12 50    .P
=> i2c md 0x4c 2 2
0002: 4b 00    K.
=> i2c md 0x4c 3 2
0003: 50 00    P.
=> i2c md 0x4c 1 1
0001: 28    (
diff mbox series

Patch

diff --git a/Documentation/hwmon/lm75.rst b/Documentation/hwmon/lm75.rst
index 6adab608dd05..9877ddbbf7c8 100644
--- a/Documentation/hwmon/lm75.rst
+++ b/Documentation/hwmon/lm75.rst
@@ -121,9 +121,9 @@  Supported chips:
 
          https://www.ti.com/product/TMP1075
 
-  * NXP LM75B, PCT2075
+  * NXP LM75B, P3T1755, PCT2075
 
-    Prefix: 'lm75b', 'pct2075'
+    Prefix: 'lm75b', 'p3t1755', 'pct2075'
 
     Addresses scanned: none
 
@@ -131,6 +131,8 @@  Supported chips:
 
 	       https://www.nxp.com/documents/data_sheet/LM75B.pdf
 
+               https://www.nxp.com/docs/en/data-sheet/P3T1755.pdf
+
                https://www.nxp.com/docs/en/data-sheet/PCT2075.pdf
 
   * AMS OSRAM AS6200
diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
index 2c2205aec7d4..a8f95bef68cb 100644
--- a/drivers/hwmon/lm75.c
+++ b/drivers/hwmon/lm75.c
@@ -38,6 +38,7 @@  enum lm75_type {		/* keep sorted in alphabetical order */
 	max6626,
 	max31725,
 	mcp980x,
+	p3t1755,
 	pct2075,
 	stds75,
 	stlm75,
@@ -222,6 +223,13 @@  static const struct lm75_params device_params[] = {
 		.default_resolution = 9,
 		.default_sample_time = MSEC_PER_SEC / 18,
 	},
+	[p3t1755] = {
+		.clr_mask = 1 << 1 | 1 << 7,	/* disable SMBAlert and one-shot */
+		.default_resolution = 12,
+		.default_sample_time = 55,
+		.num_sample_times = 4,
+		.sample_times = (unsigned int []){ 28, 55, 110, 220 },
+	},
 	[pct2075] = {
 		.default_resolution = 11,
 		.default_sample_time = MSEC_PER_SEC / 10,
@@ -734,6 +742,7 @@  static const struct i2c_device_id lm75_ids[] = {
 	{ "max31725", max31725, },
 	{ "max31726", max31725, },
 	{ "mcp980x", mcp980x, },
+	{ "p3t1755", p3t1755, },
 	{ "pct2075", pct2075, },
 	{ "stds75", stds75, },
 	{ "stlm75", stlm75, },
@@ -813,6 +822,10 @@  static const struct of_device_id __maybe_unused lm75_of_match[] = {
 		.compatible = "maxim,mcp980x",
 		.data = (void *)mcp980x
 	},
+	{
+		.compatible = "nxp,p3t1755",
+		.data = (void *)p3t1755
+	},
 	{
 		.compatible = "nxp,pct2075",
 		.data = (void *)pct2075