On Thu, 21 Nov 2019, Sven Van Asbroeck wrote: > The driver has been extended to optionally get its operational > mode, regulator init data and led naming from the devicetree. s/led/LED/ > Tree: next-20191118 Why is this in your commit message? > Signed-off-by: Sven Van Asbroeck <TheSven73@xxxxxxxxx> > --- > .../devicetree/bindings/mfd/tps6105x.txt | 47 ++++++++++++++++++- > 1 file changed, 46 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/mfd/tps6105x.txt b/Documentation/devicetree/bindings/mfd/tps6105x.txt > index 93602c7a19c8..1fc9aa3ed882 100644 > --- a/Documentation/devicetree/bindings/mfd/tps6105x.txt > +++ b/Documentation/devicetree/bindings/mfd/tps6105x.txt > @@ -7,11 +7,56 @@ Required properties: > - compatible: "ti,tps61050" or "ti,tps61052" > - reg: Specifies the I2C slave address > > -Example: > +Optional sub-node: > + > +This subnode selects the chip's operational mode. > +There can be at most one single available subnode. > + > +- regulator: presence of this sub-node puts the chip in regulator mode. > + see Documentation/devicetree/bindings/regulator/regulator.txt > + > +- led: presence of this sub-node puts the chip in led mode. > + Optional properties: > + - function : see Documentation/devicetree/bindings/leds/common.txt > + - color : see Documentation/devicetree/bindings/leds/common.txt > + - label : see Documentation/devicetree/bindings/leds/common.txt > + (deprecated) Nit: Could you please use relative paths here. If you line up the ':', it would look slightly cleaner. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog