On Sat, 8 Dec 2018 16:19:59 -0200 Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx> wrote: > Add a devicetree documentation for the ad5933 and ad5934 impedance > converter, network analyzer. > > Signed-off-by: Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx> > Signed-off-by: Gabriel Capella <gabriel@xxxxxxxxxxx> > Co-Developed-by: Gabriel Capella <gabriel@xxxxxxxxxxx> Device tree binding patches need to also go to the DT maintainers and list for review. This looks fine to me but they are the specialists in these! +cc Rob, Mark and DT list. Jonathan > --- > .../iio/impedance-analyzer/ad5933.txt | 26 +++++++++++++++++++ > 1 file changed, 26 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..5ff38728ff91 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/impedance-analyzer/ad5933.txt > @@ -0,0 +1,26 @@ > +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 : The regulator supply for DVDD, AVDD1 and AVDD2 when they > + are connected together. > + > +Optional properties: > +- clocks : external clock reference. > +- clock-names : must be "mclk" if clocks is set. > + > +Example for a I2C device node: > + > + impedance-analyzer@0d { > + compatible = "adi,adxl345"; > + reg = <0x0d>; > + vdd-supply = <&vdd_supply>; > + clocks = <&ref_clk>; > + clock-names = "mclk"; > + };
