Re: [PATCH v3 2/2] NFC: pn544_i2c: Add DTS Documentation

[Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>]

On Mon, Mar 31, 2014 at 01:22:24PM +0100, Clement Perrochaud wrote:
> Describe the properties used by the PN544 NFC controller driver.
> 
> Signed-off-by: Clément Perrochaud <clement.perrochaud-3arQi8VN3Tc@public.gmane.org>
> ---
>  .../devicetree/bindings/net/nfc/pn544.txt          | 32 ++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/net/nfc/pn544.txt
> 
> diff --git a/Documentation/devicetree/bindings/net/nfc/pn544.txt b/Documentation/devicetree/bindings/net/nfc/pn544.txt
> new file mode 100644
> index 0000000..2017ad5
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/nfc/pn544.txt
> @@ -0,0 +1,32 @@
> +* NXP Semiconductors PN544 NFC Controller
> +
> +Required properties:
> +- compatible: Should be "nxp,pn544_i2c".

s/_/-/ in compatible strings and property names, please.

> +- clock-frequency: I²C work frequency.
> +- reg: address on the bus
> +- irq: Input GPIO pin used for interrupt requests from the controller

As this is an interrupt from the point of view of the device, I would
prefer this described as an interrupt (using the standard interrupts
property), with the GPIO controller described as an interrupt
controller.

> +- en: Output GPIO pin used for enabling/disabling the PN544

enable-gpios?

> +- fw:Output GPIO pin used to enter firmware download mode

firmware-gpios?

Cheers,
Mark.

> +
> +Optional SoC Specific Properties:
> +- pinctrl-names: Contains only one value - "default".
> +- pintctrl-0: Specifies the pin control groups used for this controller.
> +
> +Example (for ARM-based BeagleBone with PN544 on I2C2):
> +
> +&i2c2 {
> +
> +	status = "okay";
> +
> +    pn544: pn544@28 {
> +
> +        compatible = "nxp,pn544_i2c";
> +
> +        reg = <0x28>;
> +        clock-frequency = <400000>;
> +
> +        irq  = <&gpio1 17 GPIO_ACTIVE_HIGH>;
> +        en = <&gpio3 21 GPIO_ACTIVE_HIGH>;
> +        fw = <&gpio3 19 GPIO_ACTIVE_HIGH>;
> +    };
> +};
> -- 
> 1.9.0
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html