linux-kernel - Re: [PATCH 1/2] devantech-srf04.yaml: transform DT binding to YAML

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAL_JsqK6TAqZgqWGvdA4aaEPiZeBxug4gV6mwZoUOHB1mSZzFw@mail.gmail.com>
Date:   Tue, 26 Mar 2019 07:58:14 -0500
From:   Rob Herring <robh+dt@...nel.org>
To:     Andreas Klinger <ak@...klinger.de>
Cc:     Jonathan Cameron <jic23@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Hartmut Knaack <knaack.h@....de>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Peter Meerwald <pmeerw@...erw.net>,
        "open list:IIO SUBSYSTEM AND DRIVERS" <linux-iio@...r.kernel.org>,
        devicetree@...r.kernel.org,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] devantech-srf04.yaml: transform DT binding to YAML

On Tue, Mar 26, 2019 at 7:31 AM Andreas Klinger <ak@...klinger.de> wrote:
>
> use devantech-srf04.txt and transform binding into yaml

I'd just collapse these into 1 patch.

>
> Signed-off-by: Andreas Klinger <ak@...klinger.de>
> ---
>  .../bindings/iio/proximity/devantech-srf04.yaml    | 62 ++++++++++++++++++++++
>  1 file changed, 62 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml
>
> diff --git a/Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml b/Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml
> new file mode 100644
> index 000000000000..cd065393b062
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml
> @@ -0,0 +1,62 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/proximity/devantech-srf04.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Devantech SRF04 ultrasonic range finder
> +
> +maintainers:
> +  - Andreas Klinger <ak@...klinger.de>
> +
> +description: |
> +  Bit-banging driver using two GPIOs:
> +  - trigger-gpio is raised by the driver to start sending out an ultrasonic
> +    burst
> +  - echo-gpio is held high by the sensor after sending ultrasonic burst
> +    until it is received once again
> +
> +  Specifications about the driver can be found at:
> +  http://www.robot-electronics.co.uk/htm/srf04tech.htm
> +
> +allOf:
> +  - $ref: /schemas/i2c/i2c-controller.yaml#

This doesn't belong. It's not a I2C bus controller.

> +
> +properties:
> +  compatible:
> +    items:
> +      - const: devantech,srf04
> +
> +  trig-gpios:
> +    description:
> +      Definition of the GPIO for the triggering (output) This GPIO is set
> +      for about 10 us by the driver to tell the device it should initiate
> +      the measurement cycle.
> +    maxItems: 1
> +
> +  echo-gpios:
> +    description:
> +      Definition of the GPIO for the echo (input)
> +      This GPIO is set by the device as soon as an ultrasonic burst is sent
> +      out and reset when the first echo is received.
> +      Thus this GPIO is set while the ultrasonic waves are doing one round
> +      trip.
> +      It needs to be an GPIO which is able to deliver an interrupt because
> +      the time between two interrupts is measured in the driver.
> +      See Documentation/devicetree/bindings/gpio/gpio.txt for information
> +      on how to specify a consumer gpio.
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - trig-gpios
> +  - echo-gpios
> +
> +examples:
> +  - |
> +    #include <dt-bindings/gpio/gpio.h>
> +    proximity {
> +        compatible = "devantech,srf04";
> +        trig-gpios = <&gpio1 15 GPIO_ACTIVE_HIGH>;
> +        echo-gpios = <&gpio2  6 GPIO_ACTIVE_HIGH>;
> +    };
> --
> 2.11.0