2023-06-08 11:03:47

by Matyas, Daniel

Subject: [PATCH v11 1/2] dt-bindings: hwmon: add MAX31827

MAX31827 is a low-power temperature switch with I2C interface.

The device is a ±1°C accuracy from -40°C to +125°C
(12 bits) local temperature switch and sensor with I2C/SM-
Bus interface. The combination of small 6-bump wafer-lev-
el package (WLP) and high accuracy makes this temper-
ature sensor/switch ideal for a wide range of applications.

Signed-off-by: Daniel Matyas <[email protected]>
Reviewed-by: Rob Herring <[email protected]>
---

Change log:
v10 -> v11: No change.
v9 -> v10: Added proper change log.
v8 -> v9: Did not use '--base'.
v7 -> v8: Added '--base=[Linux 6.4-rc1 commmit]' when using 'git format-patch'
v6 -> v7: No change.
v5 -> v6: Added Reviewed-by tag.
v3 -> v5: Forgot to add Reviewed-by tag.
v2 -> v3: Made max31828 and max31829 compatible with max31827.
v1 -> v2: Resolved dt_binding_check errors. Dropped status at i2c node. Made node name generic.


.../bindings/hwmon/adi,max31827.yaml | 54 +++++++++++++++++++
MAINTAINERS | 7 +++
2 files changed, 61 insertions(+)
create mode 100644 Documentation/devicetree/bindings/hwmon/adi,max31827.yaml

diff --git a/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml b/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
new file mode 100644
index 000000000000..2dc8b07b4d3b
--- /dev/null
+++ b/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
@@ -0,0 +1,54 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/hwmon/adi,max31827.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Analog Devices MAX31827, MAX31828, MAX31829 Low-Power Temperature Switch
+
+maintainers:
+ - Daniel Matyas <[email protected]>
+
+description: |
+ Analog Devices MAX31827, MAX31828, MAX31829 Low-Power Temperature Switch with
+ I2C Interface
+ https://www.analog.com/media/en/technical-documentation/data-sheets/MAX31827-MAX31829.pdf
+
+properties:
+ compatible:
+ oneOf:
+ - const: adi,max31827
+ - items:
+ - enum:
+ - adi,max31828
+ - adi,max31829
+ - const: adi,max31827
+
+ reg:
+ maxItems: 1
+
+ vref-supply:
+ description:
+ Must have values in the interval (1.6V; 3.6V) in order for the device to
+ function correctly.
+
+required:
+ - compatible
+ - reg
+ - vref-supply
+
+additionalProperties: false
+
+examples:
+ - |
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ temperature-sensor@42 {
+ compatible = "adi,max31827";
+ reg = <0x42>;
+ vref-supply = <&reg_vdd>;
+ };
+ };
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index 27ef11624748..752c7e9c4e4a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12612,6 +12612,13 @@ F: Documentation/userspace-api/media/drivers/max2175.rst
F: drivers/media/i2c/max2175*
F: include/uapi/linux/max2175.h

+MAX31827 TEMPERATURE SWITCH DRIVER
+M: Daniel Matyas <[email protected]>
+L: [email protected]
+S: Supported
+W: http://ez.analog.com/community/linux-device-drivers
+F: Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
+
MAX6650 HARDWARE MONITOR AND FAN CONTROLLER DRIVER
L: [email protected]
S: Orphan
--
2.34.1

2023-06-08 11:51:58

by Krzysztof Kozlowski

Subject: Re: [PATCH v11 1/2] dt-bindings: hwmon: add MAX31827

On 08/06/2023 12:41, Daniel Matyas wrote:
> MAX31827 is a low-power temperature switch with I2C interface.
>
> The device is a ±1°C accuracy from -40°C to +125°C
> (12 bits) local temperature switch and sensor with I2C/SM-
> Bus interface. The combination of small 6-bump wafer-lev-
> el package (WLP) and high accuracy makes this temper-
> ature sensor/switch ideal for a wide range of applications.
>
> Signed-off-by: Daniel Matyas <[email protected]>
> Reviewed-by: Rob Herring <[email protected]>
> ---
>
> Change log:
> v10 -> v11: No change.

Wasn't this applied? Why do you send it again?

Best regards,
Krzysztof

2023-06-08 13:49:47

by Guenter Roeck

Subject: Re: [PATCH v11 1/2] dt-bindings: hwmon: add MAX31827

On 6/8/23 04:30, Krzysztof Kozlowski wrote:
> On 08/06/2023 12:41, Daniel Matyas wrote:
>> MAX31827 is a low-power temperature switch with I2C interface.
>>
>> The device is a ±1°C accuracy from -40°C to +125°C
>> (12 bits) local temperature switch and sensor with I2C/SM-
>> Bus interface. The combination of small 6-bump wafer-lev-
>> el package (WLP) and high accuracy makes this temper-
>> ature sensor/switch ideal for a wide range of applications.
>>
>> Signed-off-by: Daniel Matyas <[email protected]>
>> Reviewed-by: Rob Herring <[email protected]>
>> ---
>>
>> Change log:
>> v10 -> v11: No change.
>
> Wasn't this applied? Why do you send it again?
>

There was a problem reported in the driver implementation
(bad compatible entry). The submitter chose to send
v11 instead of a patch fixing the already applied driver.

I'll fix up the original driver patch instead. Please ignore.

Guenter