Felipe On 08/22/2014 09:15 AM, Balbi, Felipe wrote: > On Fri, Aug 22, 2014 at 09:12:21AM -0500, Dan Murphy wrote: >> Update the drv260x dt binding document: >> - Change the node name to the devices function not the >> device name. >> - Add vbat-supply to the example. >> - Fix indentation of the example. >> >> Signed-off-by: Dan Murphy <dmurphy@xxxxxx> >> --- >> .../devicetree/bindings/input/ti,drv260x.txt | 19 ++++++++++--------- >> 1 file changed, 10 insertions(+), 9 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/input/ti,drv260x.txt b/Documentation/devicetree/bindings/input/ti,drv260x.txt >> index a9c8519..214e25d 100644 >> --- a/Documentation/devicetree/bindings/input/ti,drv260x.txt >> +++ b/Documentation/devicetree/bindings/input/ti,drv260x.txt >> @@ -37,15 +37,16 @@ Optional properties: >> 3.2 v. >> Example: >> >> -drv2605l: drv2605l@5a { >> - compatible = "ti,drv2605l"; >> - reg = <0x5a>; >> - enable-gpio = <&gpio1 28 GPIO_ACTIVE_HIGH>; >> - mode = <DRV260X_LRA_MODE>; >> - library-sel = <DRV260X_LIB_LRA>; >> - vib-rated-mv = <3200>; >> - vib-overdriver-mv = <3200>; >> -}; >> +haptics:@5a { > > haptics: haptics@5a ?? > I found both types of entries in the bindings docs. There is no consistency within the input bindings documentation. I have no issue doing it either way. Dan -- ------------------ Dan Murphy -- 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