On Mon, 21 Sep 2015, Andrew F. Davis wrote: > On 09/19/2015 11:16 PM, Lee Jones wrote: > >On Tue, 15 Sep 2015, Andrew F. Davis wrote: > > > >>The TPS65912 PMIC contains several regulators and a GPIO controller. > >>Add bindings for the TPS65912 PMIC. > >> > >>Signed-off-by: Andrew F. Davis <afd@xxxxxx> > >>--- > >> .../devicetree/bindings/gpio/gpio-tps65912.txt | 17 +++++++++ > >> Documentation/devicetree/bindings/mfd/tps65912.txt | 43 ++++++++++++++++++++++ > >> .../bindings/regulator/tps65912-regulator.txt | 32 ++++++++++++++++ > >> 3 files changed, 92 insertions(+) > >> create mode 100644 Documentation/devicetree/bindings/gpio/gpio-tps65912.txt > >> create mode 100644 Documentation/devicetree/bindings/mfd/tps65912.txt > >> create mode 100644 Documentation/devicetree/bindings/regulator/tps65912-regulator.txt [...] > >>+Optional nodes: > >>+ - Regulators: Documentation/devicetree/bindings/regulator/tps65912-regulator.txt > >>+ - GPIO: Documentation/devicetree/bindings/gpio/gpio-tps65912.txt. > > > >Better to use ../gpio, ../regulator, etc. > > > >"Regulators" and "GPIO" aren't valid node names. > > > >Please be more specific. > > > > OK, I'll see if I can clear this up. [...] > >>+Optional properties: > >>+ - Any optional property defined in bindings/regulator/regulator.txt > > > >../regulator/... > > > > Not really sure what you mean here? Same as above. Use "../regulator/regulator.txt" instead. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html