On Fri, Aug 22, 2014 at 12:12:19PM -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 <dmur...@ti.com>

Reviewed-by: Felipe Balbi <ba...@ti.com>

> ---
> 
> v2 - Address comments on indentation, node declaration, add vbat-supply,
> removed description as it did not make sense - 
> https://patchwork.kernel.org/patch/4765141/
> 
>  .../devicetree/bindings/input/ti,drv260x.txt       |   23 
> ++++++++++----------
>  1 file changed, 11 insertions(+), 12 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/input/ti,drv260x.txt 
> b/Documentation/devicetree/bindings/input/ti,drv260x.txt
> index a9c8519..ee09c8f 100644
> --- a/Documentation/devicetree/bindings/input/ti,drv260x.txt
> +++ b/Documentation/devicetree/bindings/input/ti,drv260x.txt
> @@ -1,6 +1,4 @@
> -Texas Instruments - drv260x Haptics driver family
> -
> -The drv260x family serial control bus communicates through I2C protocols
> +* Texas Instruments - drv260x Haptics driver family
>  
>  Required properties:
>       - compatible - One of:
> @@ -37,15 +35,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: haptics@5a {
> +     compatible = "ti,drv2605l";
> +     reg = <0x5a>;
> +     vbat-supply = <&vbat>;
> +     enable-gpio = <&gpio1 28 GPIO_ACTIVE_HIGH>;
> +     mode = <DRV260X_LRA_MODE>;
> +     library-sel = <DRV260X_LIB_LRA>;
> +     vib-rated-mv = <3200>;
> +     vib-overdriver-mv = <3200>;
> +}
>  
>  For more product information please see the link below:
>  http://www.ti.com/product/drv2605
> -- 
> 1.7.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to