linux-kernel - Re: [PATCH] staging: iio: ad5933: add binding doc for ad5933

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 for Android: free password hash cracker in your pocket

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

Date:   Mon, 3 Dec 2018 13:16:21 +0000
From:   Jonathan Cameron <jonathan.cameron@...wei.com>
To:     Marcelo Schmitt <marcelo.schmitt1@...il.com>
CC:     <lars@...afoo.de>, <Michael.Hennerich@...log.com>,
        <jic23@...nel.org>, <knaack.h@....de>, <pmeerw@...erw.net>,
        <gregkh@...uxfoundation.org>, <linux-iio@...r.kernel.org>,
        <devel@...verdev.osuosl.org>, <linux-kernel@...r.kernel.org>,
        <kernel-usp@...glegroups.com>
Subject: Re: [PATCH] staging: iio: ad5933: add binding doc for ad5933

On Sun, 2 Dec 2018 14:57:12 -0200
Marcelo Schmitt <marcelo.schmitt1@...il.com> wrote:

> Add a devicetree documentation for the ad5933 and ad5934 impedance
> converter, network analyzer.
> 
> Co-Developed-by: Gabriel Capella <gabriel@...ella.pro>

Put this in line below Gabriel's sign off - it makes script
parsing of these easier.

Signed-off-by: Marcelo Schmitt <marcelo.schmitt1@...il.com>
Signed-off-by: Gabriel Capella <gabriel@...ella.pro>
Co-Developed-by: Gabriel Capella <gabriel@...ella.pro>

> 
> Signed-off-by: Marcelo Schmitt <marcelo.schmitt1@...il.com>
> Signed-off-by: Gabriel Capella <gabriel@...ella.pro>
> ---
>  .../iio/impedance-analyzer/ad5933.txt         | 23 +++++++++++++++++++
>  1 file changed, 23 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/impedance-analyzer/ad5933.txt
> 
> diff --git a/Documentation/devicetree/bindings/iio/impedance-analyzer/ad5933.txt b/Documentation/devicetree/bindings/iio/impedance-analyzer/ad5933.txt
> new file mode 100644
> index 000000000000..d9ae2babf016
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/impedance-analyzer/ad5933.txt
> @@ -0,0 +1,23 @@
> +Analog Devices AD5933/AD5934 Impedance Converter, Network Analyzer
> +
> +https://www.analog.com/media/en/technical-documentation/data-sheets/AD5933.pdf
> +https://www.analog.com/media/en/technical-documentation/data-sheets/AD5934.pdf
> +
> +Required properties:
> + - compatible : should be one of
> +		"adi,ad5933"
> +		"adi,ad5934"
> + - reg : the I2C address
> + - vdd-supply : supply reference for the device.
> +
> +Optional properties:
> + - vref_mv : default voltage reference.

Regulator framework rather than a simple voltage.

> + - ext_clk_hz : external master clock for the system.

Please use the clocks framework for this rather than rolling our own.
There are fixed clocks to cover the common case, but it may be supplied by
something smarter.

> +
> +Example:
> +
> +	impedance-analyzer@0d {
> +		compatible = "adi,ad5933";
> +		reg = <0x0d>;
> +		vdd-supply = <&vdd_supply>;
> +	};