netdev - Re: [PATCH net-next v2 3/3] devicetree: add devicetree bindings for cc2520 driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 18 Jun 2014 10:05:00 +0100
From:	Mark Rutland <mark.rutland@....com>
To:	Varka Bhadram <varkabhadram@...il.com>
Cc:	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"alex.bluesman.smirnov@...il.com" <alex.bluesman.smirnov@...il.com>,
	"dbaryshkov@...il.com" <dbaryshkov@...il.com>,
	"linux-zigbee-devel@...ts.sourceforge.net" 
	<linux-zigbee-devel@...ts.sourceforge.net>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"davem@...emloft.net" <davem@...emloft.net>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	Varka Bhadram <varkab@...c.in>
Subject: Re: [PATCH net-next v2 3/3] devicetree: add devicetree bindings for
 cc2520 driver

On Wed, Jun 18, 2014 at 06:16:05AM +0100, Varka Bhadram wrote:
> devicetree bindings for cc2520 radio driver.
> 
> Signed-off-by: Varka Bhadram <varkab@...c.in>
> ---
>  .../devicetree/bindings/net/ieee802154/cc2520.txt  |   26 ++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/net/ieee802154/cc2520.txt
> 
> diff --git a/Documentation/devicetree/bindings/net/ieee802154/cc2520.txt b/Documentation/devicetree/bindings/net/ieee802154/cc2520.txt
> new file mode 100644
> index 0000000..f0a749d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/ieee802154/cc2520.txt
> @@ -0,0 +1,26 @@
> +CC2520 IEEE 802.15.4 Compatible Radio
> +
> +Required properties:
> +	- compatible:		should be "ti,cc2520"
> +	- spi-max-frequency:	maximal bus speed (8000000), should be set to 4000000 depends
> +				sync or async operation mode
> +	- reg:			the chipselect index
> +	- mode:			SPI mode

What exactly is this property? It doesn't seem to be used by the driver,
the example is opaque, and this doesn't seem to match any existing
bindings (a grep for "mode" in Documentation/devicetree/bindings/spi
only found me a string "mode" property).

The example has several properties (the GPIOs and pinctrl) not listed
here...

Mark.

> +Example:
> +
> +	cc2520@0 {
> +		compatible = "ti,cc2520";
> +		reg = <0>;
> +		spi-max-frequency = <4000000>;
> +		mode = <1>;
> +
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&cc2520_cape_pins>;	/*GPIO pin muxing details*/
> +
> +		fifo-gpio = <&gpio1 18 0>;
> +		fifop-gpio = <&gpio1 19 0>;
> +		sfd-gpio = <&gpio1 13 0>;
> +		cca-gpio = <&gpio1 16 0>;
> +		vreg-gpio = <&gpio0 31 0>;
> +		reset-gpio = <&gpio1 12 0>;
> +	};
> -- 
> 1.7.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html