Re: [PATCH 6/6] [RFC] arm: shmobile: genmai reference: Add RSPI nodes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Geert,

Thank you for the patch.

On Tuesday 24 December 2013 12:56:50 Geert Uytterhoeven wrote:
> Add pinctrl and SPI devices for RSPI on Genmai.
> 
> On this board, only rspi4 is in use. It's bus contains a single device
> (a wm8978 audio codec), for which no bindings are defined yet.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx
> ---
>  arch/arm/boot/dts/r7s72100-genmai-reference.dts |   18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/r7s72100-genmai-reference.dts
> b/arch/arm/boot/dts/r7s72100-genmai-reference.dts index
> 114510f8bf09..6d99630627e4 100644
> --- a/arch/arm/boot/dts/r7s72100-genmai-reference.dts
> +++ b/arch/arm/boot/dts/r7s72100-genmai-reference.dts
> @@ -43,7 +43,7 @@
>  };
> 
>  &pfc {
> -	pinctrl-0 = <&scif2_pins &ethernet_pins>;
> +	pinctrl-0 = <&scif2_pins &ethernet_pins &rspi4_pins>;

You should add pinctrl-0 and pinctrl-names properties to the spi4 node 
instead. Device nodes should reference their pinctrl configuration directly, 
scif2 and ethernet are exceptions as DT bindings for those devices are not in 
mainline yet.

>  	pinctrl-names = "default";
> 
>  	scif2_pins: serial2 {
> @@ -73,6 +73,12 @@
>  				 "ethernet_int_p1_15";
>  		renesas,function = "ethernet";
>  	};
> +
> +	rspi4_pins: spi4 {
> +		renesas,groups = "rspi4_rspck_p4_0", "rspi4_ssl0_p4_1",
> +				 "rspi4_mosi_p4_2", "rspi4_miso_p4_3";
> +		renesas,function = "rspi4";
> +	};

This node is at the right place and doesn't need to be moved.

>  };
> 
>  &i2c2 {
> @@ -85,3 +91,13 @@
>  		pagesize = <64>;
>  	};
>  };
> +
> +&spi4 {
> +	status = "okay";
> +
> +	codec: wm8978@0 {
> +		compatible = "wlf,wm8978";
> +		reg = <0>;
> +		spi-max-frequency = <5000000>;
> +	};
> +};
-- 
Regards,

Laurent Pinchart

--
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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux