devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Slawomir Stepien <sst@poczta.fm>
To: Jonathan Cameron <jic23@kernel.org>
Cc: linux-iio@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	devicetree@vger.kernel.org,
	Jonathan Cameron <Jonathan.Cameron@huawei.com>
Subject: Re: [PATCH 03/46] dt-bindings:iio:potentiometer:microchip,mcp4131 txt to yaml conversion
Date: Mon, 2 Nov 2020 17:19:15 +0100	[thread overview]
Message-ID: <20201102161915.GC524499@t480s.localdomain> (raw)
In-Reply-To: <20201031184854.745828-4-jic23@kernel.org>

On paź 31, 2020 18:48, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> 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@huawei.com>
> Cc: Slawomir Stepien <sst@poczta.fm>

Reviewed-by: Slawomir Stepien <sst@poczta.fm>

> ---
>  .../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@poczta.fm>
> +
> +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

  reply	other threads:[~2020-11-02 16:19 UTC|newest]

Thread overview: 104+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-31 18:48 [PATCH 00/46] dt-bindings:iio: yet more txt to yam conversions Jonathan Cameron
2020-10-31 18:48 ` [PATCH 01/46] dt-bindings:iio:resolver:adi,ad2s90: Conversion of binding to yaml Jonathan Cameron
2020-10-31 18:48 ` [PATCH 02/46] dt-bindings:iio:potentiometer:adi,ad5272 yaml conversion Jonathan Cameron
2020-11-02  4:48   ` Phil Reid
2020-11-03 16:10   ` Rob Herring
2020-11-03 17:28     ` Jonathan Cameron
2020-11-04  0:39       ` Phil Reid
2020-11-04  3:12         ` Rob Herring
2020-11-04  4:53           ` Phil Reid
2020-11-09  2:56             ` Phil Reid
2020-11-22 16:35               ` Jonathan Cameron
2020-11-23 23:33                 ` Phil Reid
2020-10-31 18:48 ` [PATCH 03/46] dt-bindings:iio:potentiometer:microchip,mcp4131 txt to " Jonathan Cameron
2020-11-02 16:19   ` Slawomir Stepien [this message]
2020-11-03 16:11   ` Rob Herring
2020-10-31 18:48 ` [PATCH 04/46] dt-bindings:iio:potentiometer:microchip,mcp41010 " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 05/46] dt-bindings:iio:impedance-analyzer:adi,ad5933 " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 06/46] dt-bindings:iio:samsung,sensorhub-rinato: " Jonathan Cameron
2020-11-22 16:43   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 07/46] dt-bindings:iio:health:ti,afe4403: txt to yaml binding Jonathan Cameron
2020-11-22 17:32   ` Jonathan Cameron
2020-11-28 16:15     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 08/46] dt-bindings:iio:health:ti,afe4404: txt to yaml conversion Jonathan Cameron
2020-11-28 16:24   ` Jonathan Cameron
2020-12-18 19:42   ` Rob Herring
2020-12-30 12:21     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 09/46] dt-bindings:iio:health:maxim,max30100: " Jonathan Cameron
2020-10-31 21:41   ` Matt Ranostay
2020-11-22 16:58     ` Jonathan Cameron
2020-11-22 17:24       ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 10/46] dt-bindings:iio:health:maxim,max30102: " Jonathan Cameron
2020-10-31 21:38   ` Matt Ranostay
2020-11-22 17:29     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 11/46] dt-bindings:iio:imu:adi,adis16480: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 12/46] dt-bindings:iio:imu:st,lsm6dsx: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 13/46] dt-bindings:iio:light:avago,apds9300: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 14/46] dt-bindings:iio:light:avago,apds9960: " Jonathan Cameron
2020-10-31 21:33   ` Matt Ranostay
2020-10-31 21:35     ` Matt Ranostay
2020-11-01 14:57       ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 15/46] dt-bindings:iio:light:capella,cm36651: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 16/46] dt-bindings:iio:light:sharp,gp2ap020a00f: " Jonathan Cameron
2020-11-22 17:42   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 17/46] dt-bindings:iio:light:maxim,max44009: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 18/46] dt-bindings:iio:light:ti,opt3001: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 19/46] dt-bindings:iio:light:upisemi,us51882: " Jonathan Cameron
2020-11-03 16:18   ` Rob Herring
2020-10-31 18:48 ` [PATCH 20/46] dt-bindings:iio:light:st,uvis25: txt to yaml conversion for this UV sensor Jonathan Cameron
2020-10-31 18:48 ` [PATCH 21/46] dt-bindings:iio:light:vishay,vcnl4035: txt to yaml conversion Jonathan Cameron
2020-11-22 17:52   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 22/46] dt-bindings:iio:light:st,vl6180: txt to yaml format conversion Jonathan Cameron
2020-11-01 15:26   ` Manivannan Sadhasivam
2020-10-31 18:48 ` [PATCH 23/46] dt-bindings:iio:magnetometer:fsl,mag3110: txt to yaml conversion Jonathan Cameron
2020-10-31 18:48 ` [PATCH 24/46] dt-bindings:iio:magnetometer:asahi-kasei,ak8974: txt to yaml format conversion Jonathan Cameron
2020-11-04 14:47   ` Linus Walleij
2020-11-08 15:55     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 25/46] dt-bindings:iio:magnetometer:bosch,bmc150_magn: txt to yaml conversion Jonathan Cameron
2020-10-31 18:48 ` [PATCH 26/46] dt-bindings:iio:magnetometer:honeywell,hmc5843: txt to yaml format conversion Jonathan Cameron
2020-10-31 18:48 ` [PATCH 27/46] dt-bindings:iio:magnetometer:pni,rm3100: txt to yaml conversion Jonathan Cameron
2020-10-31 18:48 ` [PATCH 28/46] dt-bindings:iio:adc:atmel,sama5d2-adc: " Jonathan Cameron
2020-11-22 18:07   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 29/46] dt-bindings:iio:adc:atmel,sama9260-adc: conversion to yaml from at91_adc.txt Jonathan Cameron
2020-11-13  9:22   ` Alexandre Belloni
2020-11-14 15:37     ` Jonathan Cameron
2020-11-22 18:09       ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 30/46] dt-bindings:iio:adc:renesas,rcar-gyroadc: txt to yaml conversion Jonathan Cameron
2020-11-03 16:26   ` Rob Herring
2020-11-27  8:14   ` Geert Uytterhoeven
2020-10-31 18:48 ` [PATCH 31/46] dt-bindings:iio:adc:x-powers,axp209-adc: " Jonathan Cameron
2020-11-22 18:15   ` Jonathan Cameron
2020-11-23  3:02     ` Chen-Yu Tsai
2020-11-23 20:36       ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 32/46] dt-bindings:iio:adc:brcm,iproc-static-adc: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 33/46] dt-bindings:iio:adc:mediatek,mt2701-auxadc: rename and " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 34/46] dt-bindings:iio:adc:ti,palmas-gpadc: txt to yaml format conversion Jonathan Cameron
2020-11-22 18:20   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 35/46] dt-bindings:iio:adc:qcom,pm8018-adc: yaml conversion and rename Jonathan Cameron
2020-11-04 14:51   ` Linus Walleij
2020-10-31 18:48 ` [PATCH 36/46] dt-bindings:iio:adc:qcom,spmi-iadc: txt to yaml format conversion Jonathan Cameron
2020-11-22 18:25   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 37/46] dt-binding:iio:adc:ti,ads124s08: " Jonathan Cameron
2020-11-02 12:49   ` Dan Murphy
2020-11-08 16:59     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 38/46] dt-bindings:iio:dac:ad5592r: " Jonathan Cameron
2020-11-03 16:31   ` Rob Herring
2020-11-22 18:38     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 39/46] dt-bindings:iio:dac:ad5755: " Jonathan Cameron
2020-11-03 16:32   ` Rob Herring
2020-11-22 18:40     ` Jonathan Cameron
2021-02-21 16:33       ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 40/46] dt-bindings:iio:accel:bosch,bma180: " Jonathan Cameron
2020-11-22 18:44   ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 41/46] dt-bindings:iio:accel:kionix,kxcjk1013: " Jonathan Cameron
2020-10-31 18:48 ` [PATCH 42/46] dt-bindings:iio:accel:fsl,mma8452: txt to yaml conversion Jonathan Cameron
2020-11-03 16:34   ` Rob Herring
2020-11-22 18:50     ` Jonathan Cameron
2020-10-31 18:48 ` [PATCH 43/46] dt-bindings:iio:gyro:bosch,bmg180: txt to yaml format conversion Jonathan Cameron
2020-10-31 18:48 ` [PATCH 44/46] dt-bindings:iio:st,st-sensors: txt to yaml conversion Jonathan Cameron
2020-11-02 16:16   ` Denis CIOCCA
2020-10-31 18:48 ` [PATCH 45/46] dt-bindings:iio:frequency:adi,adf4350: txt to yaml format conversion Jonathan Cameron
2020-10-31 18:48 ` [PATCH 46/46] dt-bindings:iio:temperature: Drop generic binding file Jonathan Cameron
2020-11-03 16:38 ` [PATCH 00/46] dt-bindings:iio: yet more txt to yam conversions Rob Herring
2020-11-03 17:24   ` Jonathan Cameron
2020-11-22 18:59     ` Jonathan Cameron

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201102161915.GC524499@t480s.localdomain \
    --to=sst@poczta.fm \
    --cc=Jonathan.Cameron@huawei.com \
    --cc=devicetree@vger.kernel.org \
    --cc=jic23@kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).