Re: [PATCH 5/6] dt-bindings: hwmon: jedec,jc42: convert to dtschema

[Guenter Roeck <linux@roeck-us.net>]

On Mon, Sep 20, 2021 at 08:21:13PM +0200, Krzysztof Kozlowski wrote:
> Convert the Jedec JC-42.4 temperature sensor bindings to DT schema
> format.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>

Applied.

Thanks,
Guenter

> ---
>  .../devicetree/bindings/hwmon/jc42.txt        | 46 -------------
>  .../devicetree/bindings/hwmon/jedec,jc42.yaml | 69 +++++++++++++++++++
>  MAINTAINERS                                   |  1 +
>  3 files changed, 70 insertions(+), 46 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/hwmon/jc42.txt
>  create mode 100644 Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
> 
> diff --git a/Documentation/devicetree/bindings/hwmon/jc42.txt b/Documentation/devicetree/bindings/hwmon/jc42.txt
> deleted file mode 100644
> index f569db58f64a..000000000000
> --- a/Documentation/devicetree/bindings/hwmon/jc42.txt
> +++ /dev/null
> @@ -1,46 +0,0 @@
> -Properties for Jedec JC-42.4 compatible temperature sensors
> -
> -Required properties:
> -- compatible: May include a device-specific string consisting of the
> -	      manufacturer and the name of the chip. A list of supported
> -	      chip names follows.
> -	      Must include "jedec,jc-42.4-temp" for any Jedec JC-42.4
> -	      compatible temperature sensor.
> -
> -	      Supported chip names:
> -		adi,adt7408
> -		atmel,at30ts00
> -		atmel,at30tse004
> -		onnn,cat6095
> -		onnn,cat34ts02
> -		maxim,max6604
> -		microchip,mcp9804
> -		microchip,mcp9805
> -		microchip,mcp9808
> -		microchip,mcp98243
> -		microchip,mcp98244
> -		microchip,mcp9843
> -		nxp,se97
> -		nxp,se98
> -		st,stts2002
> -		st,stts2004
> -		st,stts3000
> -		st,stts424
> -		st,stts424e
> -		idt,tse2002
> -		idt,tse2004
> -		idt,ts3000
> -		idt,ts3001
> -
> -- reg: I2C address
> -
> -Optional properties:
> -- smbus-timeout-disable: When set, the smbus timeout function will be disabled.
> -			 This is not supported on all chips.
> -
> -Example:
> -
> -temp-sensor@1a {
> -	compatible = "jedec,jc-42.4-temp";
> -	reg = <0x1a>;
> -};
> diff --git a/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml b/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
> new file mode 100644
> index 000000000000..a7bb4e3a1c46
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
> @@ -0,0 +1,69 @@
> +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/hwmon/jedec,jc42.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Jedec JC-42.4 compatible temperature sensors
> +
> +maintainers:
> +  - Jean Delvare <jdelvare@suse.com>
> +  - Guenter Roeck <linux@roeck-us.net>
> +
> +properties:
> +  compatible:
> +    oneOf:
> +      - const: jedec,jc-42.4-temp
> +      - items:
> +          - enum:
> +              - adi,adt7408
> +              - atmel,at30ts00
> +              - atmel,at30tse004
> +              - idt,tse2002
> +              - idt,tse2004
> +              - idt,ts3000
> +              - idt,ts3001
> +              - maxim,max6604
> +              - microchip,mcp9804
> +              - microchip,mcp9805
> +              - microchip,mcp9808
> +              - microchip,mcp98243
> +              - microchip,mcp98244
> +              - microchip,mcp9843
> +              - nxp,se97
> +              - nxp,se98
> +              - onnn,cat6095
> +              - onnn,cat34ts02
> +              - st,stts2002
> +              - st,stts2004
> +              - st,stts3000
> +              - st,stts424
> +              - st,stts424e
> +          - const: jedec,jc-42.4-temp
> +
> +  reg:
> +    maxItems: 1
> +
> +  smbus-timeout-disable:
> +    description: |
> +      When set, the smbus timeout function will be disabled. This is not
> +      supported on all chips.
> +    type: boolean
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        temp-sensor@1a {
> +            compatible = "jedec,jc-42.4-temp";
> +            reg = <0x1a>;
> +        };
> +    };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b07679009af2..15c4d3c809e8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10033,6 +10033,7 @@ JC42.4 TEMPERATURE SENSOR DRIVER
>  M:	Guenter Roeck <linux@roeck-us.net>
>  L:	linux-hwmon@vger.kernel.org
>  S:	Maintained
> +F:	Documentation/devicetree/bindings/hwmon/jedec,jc42.yaml
>  F:	Documentation/hwmon/jc42.rst
>  F:	drivers/hwmon/jc42.c
>