Re: [PATCH 3/7] dt-bindings:iio:potentiometer:maxim,ds1803 move to trivial devices.

[Slawomir Stepien <sst@poczta.fm>]

On paź 31, 2020 18:29, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Simple binding where there is no obvious benefit in maintaining a
> separate file.  Hence document in trivial-devices.yaml and drop
> the txt file.
> 
> 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/ds1803.txt     | 21 -------------------
>  .../devicetree/bindings/trivial-devices.yaml  |  6 ++++++
>  2 files changed, 6 insertions(+), 21 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/potentiometer/ds1803.txt b/Documentation/devicetree/bindings/iio/potentiometer/ds1803.txt
> deleted file mode 100644
> index df77bf552656..000000000000
> --- a/Documentation/devicetree/bindings/iio/potentiometer/ds1803.txt
> +++ /dev/null
> @@ -1,21 +0,0 @@
> -* Maxim Integrated DS1803 digital potentiometer driver
> -
> -The node for this driver must be a child node of a I2C controller, hence
> -all mandatory properties for your controller must be specified. See directory:
> -
> -        Documentation/devicetree/bindings/i2c
> -
> -for more details.
> -
> -Required properties:
> -	- compatible:  	Must be one of the following, depending on the
> -			model:
> -			"maxim,ds1803-010",
> -			"maxim,ds1803-050",
> -			"maxim,ds1803-100"
> -
> -Example:
> -ds1803: ds1803@1 {
> -	reg = <0x28>;
> -	compatible = "maxim,ds1803-010";
> -};
> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> index 10f0afd44684..695b1e379238 100644
> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> @@ -110,6 +110,12 @@ properties:
>            - isil,isl68137
>              # 5 Bit Programmable, Pulse-Width Modulator
>            - maxim,ds1050
> +            # 10 kOhm digital potentiometer with I2C interface
> +          - maxim,ds1803-010
> +            # 50 kOhm digital potentiometer with I2C interface
> +          - maxim,ds1803-050
> +            # 100 kOhm digital potentiometer with I2C interface
> +          - maxim,ds1803-100
>              # Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
>            - maxim,max1237
>              # PECI-to-I2C translator for PECI-to-SMBus/I2C protocol conversion

-- 
Slawomir Stepien