On Wed, Nov 14, 2018 at 11:01:41AM +0100, Slawomir Stepien wrote: > On lis 14, 2018 09:52, Chris Coffey wrote: > > This patch adds device tree documentation for the Microchip > > MCP41xxx/42xxx family of digital potentiometers. > > Hi > > One note inline. > Hi Stepien, Thank you for the reviews. One comment below. > > Signed-off-by: Chris Coffey <cmc@xxxxxxxxxxxxx> > > --- > > .../bindings/iio/potentiometer/mcp41010.txt | 29 ++++++++++++++++++++++ > > 1 file changed, 29 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/iio/potentiometer/mcp41010.txt > > > > diff --git a/Documentation/devicetree/bindings/iio/potentiometer/mcp41010.txt b/Documentation/devicetree/bindings/iio/potentiometer/mcp41010.txt > > new file mode 100644 > > index 0000000000..17565acace > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/iio/potentiometer/mcp41010.txt > > @@ -0,0 +1,29 @@ > > +* Microchip MCP41010/41050/41100/42010/42050/42100 Digital Potentiometer > > + driver > > + > > +Datasheet publicly available at: > > +http://ww1.microchip.com/downloads/en/devicedoc/11195c.pdf > > + > > +The node for this driver must be a child node of a SPI controller, hence > > +all mandatory properties described in > > + > > + Documentation/devicetree/bindings/spi/spi-bus.txt > > + > > +must be specified. > > + > > +Required properties: > > + - compatible: Must be one of the following, depending on the > > + model: > > + "microchip,mcp41010" > > + "microchip,mcp41050" > > + "microchip,mcp41100" > > + "microchip,mcp42010" > > + "microchip,mcp42050" > > + "microchip,mcp42100" > > + > > +Example: > > +mcp41010: potentiometer@0 { > > + compatible = "mcp41010"; > > Missing company name in compatible. > Gah! A classic case of my mind seeing what it wanted to see, instead of what was actually there. > > + reg = <0>; > > + spi-max-frequency = <500000>; > > +}; > > -- > Slawomir Stepien
