On Sat, 31 Oct 2020 18:48:36 +0000
Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
>
> Whilst this binding has a lot of elements they are all fairly standard.
> Hence pretty much direct txt to yaml line by line conversion.
>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> Cc: Ludovic Desroches <ludovic.desroches@xxxxxxxxx>
> Cc: Eugen Hristev <eugen.hristev@xxxxxxxxxxxxx>
Applied with -hz and -ms $ref dropped as per Rob's suggestion.
Thanks,
Jonathan
> ---
> .../bindings/iio/adc/at91-sama5d2_adc.txt | 50 ---------
> .../bindings/iio/adc/atmel,sama5d2-adc.yaml | 104 ++++++++++++++++++
> 2 files changed, 104 insertions(+), 50 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/at91-sama5d2_adc.txt b/Documentation/devicetree/bindings/iio/adc/at91-sama5d2_adc.txt
> deleted file mode 100644
> index 07c59f301b31..000000000000
> --- a/Documentation/devicetree/bindings/iio/adc/at91-sama5d2_adc.txt
> +++ /dev/null
> @@ -1,50 +0,0 @@
> -* AT91 SAMA5D2 Analog to Digital Converter (ADC)
> -
> -Required properties:
> - - compatible: Should be "atmel,sama5d2-adc" or "microchip,sam9x60-adc".
> - - reg: Should contain ADC registers location and length.
> - - interrupts: Should contain the IRQ line for the ADC.
> - - clocks: phandle to device clock.
> - - clock-names: Must be "adc_clk".
> - - vref-supply: Supply used as reference for conversions.
> - - vddana-supply: Supply for the adc device.
> - - atmel,min-sample-rate-hz: Minimum sampling rate, it depends on SoC.
> - - atmel,max-sample-rate-hz: Maximum sampling rate, it depends on SoC.
> - - atmel,startup-time-ms: Startup time expressed in ms, it depends on SoC.
> - - atmel,trigger-edge-type: One of possible edge types for the ADTRG hardware
> - trigger pin. When the specific edge type is detected, the conversion will
> - start. Possible values are rising, falling, or both.
> - This property uses the IRQ edge types values: IRQ_TYPE_EDGE_RISING ,
> - IRQ_TYPE_EDGE_FALLING or IRQ_TYPE_EDGE_BOTH
> -
> -Optional properties:
> - - dmas: Phandle to dma channel for the ADC.
> - - dma-names: Must be "rx" when dmas property is being used.
> - See ../../dma/dma.txt for details.
> - - #io-channel-cells: in case consumer drivers are attached, this must be 1.
> - See <Documentation/devicetree/bindings/iio/iio-bindings.txt> for details.
> -
> -Properties for consumer drivers:
> - - Consumer drivers can be connected to this producer device, as specified
> - in <Documentation/devicetree/bindings/iio/iio-bindings.txt>
> - - Channels exposed are specified in:
> - <dt-bindings/iio/adc/at91-sama5d2_adc.txt>
> -
> -Example:
> -
> -adc: adc@fc030000 {
> - compatible = "atmel,sama5d2-adc";
> - reg = <0xfc030000 0x100>;
> - interrupts = <40 IRQ_TYPE_LEVEL_HIGH 7>;
> - clocks = <&adc_clk>;
> - clock-names = "adc_clk";
> - atmel,min-sample-rate-hz = <200000>;
> - atmel,max-sample-rate-hz = <20000000>;
> - atmel,startup-time-ms = <4>;
> - vddana-supply = <&vdd_3v3_lp_reg>;
> - vref-supply = <&vdd_3v3_lp_reg>;
> - atmel,trigger-edge-type = <IRQ_TYPE_EDGE_BOTH>;
> - dmas = <&dma0 (AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1) | AT91_XDMAC_DT_PERID(25))>;
> - dma-names = "rx";
> - #io-channel-cells = <1>;
> -}
> diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> new file mode 100644
> index 000000000000..9b282c3ee8fb
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> @@ -0,0 +1,104 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/adc/atmel,sama5d2-adc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: AT91 SAMA5D2 Analog to Digital Converter (ADC)
> +
> +maintainers:
> + - Ludovic Desroches <ludovic.desroches@xxxxxxxxx>
> + - Eugen Hristev <eugen.hristev@xxxxxxxxxxxxx>
> +
> +properties:
> + compatible:
> + enum:
> + - atmel,sama5d2-adc
> + - microchip,sam9x60-adc
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + clock-names:
> + const: adc_clk
> +
> + vref-supply: true
> + vddana-supply: true
> +
> + atmel,min-sample-rate-hz:
> + $ref: '/schemas/types.yaml#/definitions/uint32'
> + description: Minimum sampling rate, it depends on SoC.
> +
> + atmel,max-sample-rate-hz:
> + $ref: '/schemas/types.yaml#/definitions/uint32'
> + description: Maximum sampling rate, it depends on SoC.
> +
> + atmel,startup-time-ms:
> + $ref: '/schemas/types.yaml#/definitions/uint32'
> + description: Startup time expressed in ms, it depends on SoC.
> +
> + atmel,trigger-edge-type:
> + $ref: '/schemas/types.yaml#/definitions/uint32'
> + description:
> + One of possible edge types for the ADTRG hardware trigger pin.
> + When the specific edge type is detected, the conversion will
> + start. Should be one of IRQ_TYPE_EDGE_RISING, IRQ_TYPE_EDGE_FALLING
> + or IRQ_TYPE_EDGE_BOTH.
> + enum: [1, 2, 3]
> +
> + dmas:
> + maxItems: 1
> +
> + dma-names:
> + const: rx
> +
> + "#io-channel-cells":
> + const: 1
> +
> +additionalProperties: false
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - clocks
> + - clock-names
> + - vref-supply
> + - vddana-supply
> + - atmel,min-sample-rate-hz
> + - atmel,max-sample-rate-hz
> + - atmel,startup-time-ms
> + - atmel,trigger-edge-type
> +
> +examples:
> + - |
> + #include <dt-bindings/dma/at91.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + soc {
> + #address-cells = <1>;
> + #size-cells = <1>;
> +
> + adc@fc030000 {
> + compatible = "atmel,sama5d2-adc";
> + reg = <0xfc030000 0x100>;
> + interrupts = <40 IRQ_TYPE_LEVEL_HIGH 7>;
> + clocks = <&adc_clk>;
> + clock-names = "adc_clk";
> + atmel,min-sample-rate-hz = <200000>;
> + atmel,max-sample-rate-hz = <20000000>;
> + atmel,startup-time-ms = <4>;
> + vddana-supply = <&vdd_3v3_lp_reg>;
> + vref-supply = <&vdd_3v3_lp_reg>;
> + atmel,trigger-edge-type = <IRQ_TYPE_EDGE_BOTH>;
> + dmas = <&dma0 (AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1) | AT91_XDMAC_DT_PERID(25))>;
> + dma-names = "rx";
> + #io-channel-cells = <1>;
> + };
> + };
> +...
