Re: [PATCH 03/46] dt-bindings:iio:potentiometer:microchip,mcp4131 txt to yaml conversion

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On paź 31, 2020 18:48, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> 
> This binding is very simple, but I think the very large number of
> compatible values make it unsuitable for moving to trivial-devices.yaml.
> Main change in the conversion was reordering the compatible list to
> numerical order.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> Cc: Slawomir Stepien <sst@xxxxxxxxx>

Reviewed-by: Slawomir Stepien <sst@xxxxxxxxx>

> ---
>  .../bindings/iio/potentiometer/mcp4131.txt    |  84 ---------------
>  .../iio/potentiometer/microchip,mcp4131.yaml  | 102 ++++++++++++++++++
>  2 files changed, 102 insertions(+), 84 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/potentiometer/mcp4131.txt b/Documentation/devicetree/bindings/iio/potentiometer/mcp4131.txt
> deleted file mode 100644
> index 3ccba16f7035..000000000000
> --- a/Documentation/devicetree/bindings/iio/potentiometer/mcp4131.txt
> +++ /dev/null
> @@ -1,84 +0,0 @@
> -* Microchip MCP413X/414X/415X/416X/423X/424X/425X/426X Digital Potentiometer
> -  driver
> -
> -The node for this driver must be a child node of a SPI controller, hence
> -all mandatory properties described in
> -
> -        Documentation/devicetree/bindings/spi/spi-bus.txt
> -
> -must be specified.
> -
> -Required properties:
> -	- compatible:  	Must be one of the following, depending on the
> -			model:
> -			"microchip,mcp4131-502"
> -			"microchip,mcp4131-103"
> -			"microchip,mcp4131-503"
> -			"microchip,mcp4131-104"
> -			"microchip,mcp4132-502"
> -			"microchip,mcp4132-103"
> -			"microchip,mcp4132-503"
> -			"microchip,mcp4132-104"
> -			"microchip,mcp4141-502"
> -			"microchip,mcp4141-103"
> -			"microchip,mcp4141-503"
> -			"microchip,mcp4141-104"
> -			"microchip,mcp4142-502"
> -			"microchip,mcp4142-103"
> -			"microchip,mcp4142-503"
> -			"microchip,mcp4142-104"
> -			"microchip,mcp4151-502"
> -			"microchip,mcp4151-103"
> -			"microchip,mcp4151-503"
> -			"microchip,mcp4151-104"
> -			"microchip,mcp4152-502"
> -			"microchip,mcp4152-103"
> -			"microchip,mcp4152-503"
> -			"microchip,mcp4152-104"
> -			"microchip,mcp4161-502"
> -			"microchip,mcp4161-103"
> -			"microchip,mcp4161-503"
> -			"microchip,mcp4161-104"
> -			"microchip,mcp4162-502"
> -			"microchip,mcp4162-103"
> -			"microchip,mcp4162-503"
> -			"microchip,mcp4162-104"
> -			"microchip,mcp4231-502"
> -			"microchip,mcp4231-103"
> -			"microchip,mcp4231-503"
> -			"microchip,mcp4231-104"
> -			"microchip,mcp4232-502"
> -			"microchip,mcp4232-103"
> -			"microchip,mcp4232-503"
> -			"microchip,mcp4232-104"
> -			"microchip,mcp4241-502"
> -			"microchip,mcp4241-103"
> -			"microchip,mcp4241-503"
> -			"microchip,mcp4241-104"
> -			"microchip,mcp4242-502"
> -			"microchip,mcp4242-103"
> -			"microchip,mcp4242-503"
> -			"microchip,mcp4242-104"
> -			"microchip,mcp4251-502"
> -			"microchip,mcp4251-103"
> -			"microchip,mcp4251-503"
> -			"microchip,mcp4251-104"
> -			"microchip,mcp4252-502"
> -			"microchip,mcp4252-103"
> -			"microchip,mcp4252-503"
> -			"microchip,mcp4252-104"
> -			"microchip,mcp4261-502"
> -			"microchip,mcp4261-103"
> -			"microchip,mcp4261-503"
> -			"microchip,mcp4261-104"
> -			"microchip,mcp4262-502"
> -			"microchip,mcp4262-103"
> -			"microchip,mcp4262-503"
> -			"microchip,mcp4262-104"
> -
> -Example:
> -mcp4131: mcp4131@0 {
> -	compatible = "mcp4131-502";
> -	reg = <0>;
> -	spi-max-frequency = <500000>;
> -};
> diff --git a/Documentation/devicetree/bindings/iio/potentiometer/microchip,mcp4131.yaml b/Documentation/devicetree/bindings/iio/potentiometer/microchip,mcp4131.yaml
> new file mode 100644
> index 000000000000..834d17ae3730
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/potentiometer/microchip,mcp4131.yaml
> @@ -0,0 +1,102 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/potentiometer/microchip,mcp4131.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Microchip MCP413X/414X/415X/416X/423X/424X/425X/426X Digital Potentiometer
> +maintainers:
> +  - Slawomir Stepien <sst@xxxxxxxxx>
> +
> +properties:
> +  compatible:
> +    enum:
> +      - microchip,mcp4131-103
> +      - microchip,mcp4131-104
> +      - microchip,mcp4131-502
> +      - microchip,mcp4131-503
> +      - microchip,mcp4132-103
> +      - microchip,mcp4132-104
> +      - microchip,mcp4132-502
> +      - microchip,mcp4132-503
> +      - microchip,mcp4141-103
> +      - microchip,mcp4141-104
> +      - microchip,mcp4141-502
> +      - microchip,mcp4141-503
> +      - microchip,mcp4142-103
> +      - microchip,mcp4142-104
> +      - microchip,mcp4142-502
> +      - microchip,mcp4142-503
> +      - microchip,mcp4151-103
> +      - microchip,mcp4151-104
> +      - microchip,mcp4151-502
> +      - microchip,mcp4151-503
> +      - microchip,mcp4152-103
> +      - microchip,mcp4152-104
> +      - microchip,mcp4152-502
> +      - microchip,mcp4152-503
> +      - microchip,mcp4161-103
> +      - microchip,mcp4161-104
> +      - microchip,mcp4161-502
> +      - microchip,mcp4161-503
> +      - microchip,mcp4162-103
> +      - microchip,mcp4162-104
> +      - microchip,mcp4162-502
> +      - microchip,mcp4162-503
> +      - microchip,mcp4231-103
> +      - microchip,mcp4231-104
> +      - microchip,mcp4231-502
> +      - microchip,mcp4231-503
> +      - microchip,mcp4232-103
> +      - microchip,mcp4232-104
> +      - microchip,mcp4232-502
> +      - microchip,mcp4232-503
> +      - microchip,mcp4241-103
> +      - microchip,mcp4241-104
> +      - microchip,mcp4241-502
> +      - microchip,mcp4241-503
> +      - microchip,mcp4242-103
> +      - microchip,mcp4242-104
> +      - microchip,mcp4242-502
> +      - microchip,mcp4242-503
> +      - microchip,mcp4251-103
> +      - microchip,mcp4251-104
> +      - microchip,mcp4251-502
> +      - microchip,mcp4251-503
> +      - microchip,mcp4252-103
> +      - microchip,mcp4252-104
> +      - microchip,mcp4252-502
> +      - microchip,mcp4252-503
> +      - microchip,mcp4261-103
> +      - microchip,mcp4261-104
> +      - microchip,mcp4261-502
> +      - microchip,mcp4261-503
> +      - microchip,mcp4262-103
> +      - microchip,mcp4262-104
> +      - microchip,mcp4262-502
> +      - microchip,mcp4262-503
> +
> +  reg:
> +    maxItems: 1
> +
> +  spi-max-frequency: true
> +
> +additionalProperties: false
> +
> +required:
> +  - compatible
> +  - reg
> +
> +examples:
> +  - |
> +    spi {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        potentiometer@0 {
> +            compatible = "mcp4131-502";
> +            reg = <0>;
> +            spi-max-frequency = <500000>;
> +        };
> +    };
> +...

-- 
Slawomir Stepien



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux