Felipe

On 08/22/2014 09:45 AM, Balbi, Felipe wrote:

Hi,

On Fri, Aug 22, 2014 at 09:25:13AM -0500, Murphy, Dan wrote:

quoted

Felipe

On 08/22/2014 09:15 AM, Balbi, Felipe wrote:

quoted

On Fri, Aug 22, 2014 at 09:12:21AM -0500, Dan Murphy wrote:

quoted

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 <redacted>
---
 .../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 at 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 at 5a ??

I found both types of entries in the bindings docs.
There is no consistency within the input bindings documentation.

I could swear neither two types have a colon character between node name
and its base address.

I will add it to the fixes.

Dan

-- 
------------------
Dan Murphy