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
