linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino.
  2015-04-29  8:21 [PATCH 0/4] A10s Olinuxino Micro DT SPI patches Michal Suchanek
@ 2015-04-29  7:41 ` Michal Suchanek
  2015-05-01 20:24   ` Maxime Ripard
  2015-04-29  7:59 ` [PATCH 3/4] ARM: dts: sunxi: A10s Olinuxino Micro: add spi2 Michal Suchanek
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 7+ messages in thread
From: Michal Suchanek @ 2015-04-29  7:41 UTC (permalink / raw)
  To: linux-arm-kernel

SPI aliases give nicer spidev device node names.

Signed-off-by: Michal Suchanek <hramrach@gmail.com>
---
 arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts b/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
index cd8d08a..a49fc6a 100644
--- a/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
+++ b/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
@@ -63,6 +63,9 @@
 		serial0 = &uart0;
 		serial1 = &uart2;
 		serial2 = &uart3;
+		spi0 = &spi0;
+		spi1 = &spi1;
+		spi2 = &spi2;
 	};
 
 	leds {
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/4] ARM: dts: sunxi: A10s Olinuxino Micro: add spi2.
  2015-04-29  8:21 [PATCH 0/4] A10s Olinuxino Micro DT SPI patches Michal Suchanek
  2015-04-29  7:41 ` [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino Michal Suchanek
@ 2015-04-29  7:59 ` Michal Suchanek
  2015-04-29  8:00 ` [PATCH 1/4] ARM: dts: sunxi: sun5i: add spi2_pins_b Michal Suchanek
  2015-04-29  8:02 ` [PATCH 2/4] ARM: dts: sunxi: sun5i-a10s: add spi2_pins_a Michal Suchanek
  3 siblings, 0 replies; 7+ messages in thread
From: Michal Suchanek @ 2015-04-29  7:59 UTC (permalink / raw)
  To: linux-arm-kernel

spi2 is exposed on the UEXT connector.

Signed-off-by: Michal Suchanek <hramrach@gmail.com>
---
 arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts b/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
index aa88ee8..cd8d08a 100644
--- a/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
+++ b/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
@@ -182,6 +182,12 @@
 	status = "okay";
 };
 
+&spi2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&spi2_pins_a>;
+	status = "okay";
+};
+
 &ohci0 {
 	status = "okay";
 };
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 1/4] ARM: dts: sunxi: sun5i: add spi2_pins_b.
  2015-04-29  8:21 [PATCH 0/4] A10s Olinuxino Micro DT SPI patches Michal Suchanek
  2015-04-29  7:41 ` [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino Michal Suchanek
  2015-04-29  7:59 ` [PATCH 3/4] ARM: dts: sunxi: A10s Olinuxino Micro: add spi2 Michal Suchanek
@ 2015-04-29  8:00 ` Michal Suchanek
  2015-05-01 20:22   ` Maxime Ripard
  2015-04-29  8:02 ` [PATCH 2/4] ARM: dts: sunxi: sun5i-a10s: add spi2_pins_a Michal Suchanek
  3 siblings, 1 reply; 7+ messages in thread
From: Michal Suchanek @ 2015-04-29  8:00 UTC (permalink / raw)
  To: linux-arm-kernel

spi2_pins_a are only available on A10s

Signed-off-by: Michal Suchanek <hramrach@gmail.com>
---
 arch/arm/boot/dts/sun5i.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi
index 186d25f..9f22c28 100644
--- a/arch/arm/boot/dts/sun5i.dtsi
+++ b/arch/arm/boot/dts/sun5i.dtsi
@@ -551,6 +551,13 @@
 				allwinner,drive = <SUN4I_PINCTRL_30_MA>;
 				allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
 			};
+
+			spi2_pins_b: spi2 at 1 {
+				allwinner,pins = "PE00", "PE01", "PE02", "PE03";
+				allwinner,function = "spi2";
+				allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+				allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+			};
 		};
 
 		timer at 01c20c00 {
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/4] ARM: dts: sunxi: sun5i-a10s: add spi2_pins_a.
  2015-04-29  8:21 [PATCH 0/4] A10s Olinuxino Micro DT SPI patches Michal Suchanek
                   ` (2 preceding siblings ...)
  2015-04-29  8:00 ` [PATCH 1/4] ARM: dts: sunxi: sun5i: add spi2_pins_b Michal Suchanek
@ 2015-04-29  8:02 ` Michal Suchanek
  3 siblings, 0 replies; 7+ messages in thread
From: Michal Suchanek @ 2015-04-29  8:02 UTC (permalink / raw)
  To: linux-arm-kernel

Extra SPI pins on A10s not shared with A13.

Signed-off-by: Michal Suchanek <hramrach@gmail.com>
---
 arch/arm/boot/dts/sun5i-a10s.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/sun5i-a10s.dtsi b/arch/arm/boot/dts/sun5i-a10s.dtsi
index a78c95d..3cdd122 100644
--- a/arch/arm/boot/dts/sun5i-a10s.dtsi
+++ b/arch/arm/boot/dts/sun5i-a10s.dtsi
@@ -198,4 +198,12 @@
 		allwinner,drive = <SUN4I_PINCTRL_30_MA>;
 		allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
 	};
+
+	spi2_pins_a: spi2 at 0 {
+		allwinner,pins = "PB11", "PB12", "PB13", "PB14";
+		allwinner,function = "spi2";
+		allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+		allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+	};
+
 };
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 0/4] A10s Olinuxino Micro DT SPI patches
@ 2015-04-29  8:21 Michal Suchanek
  2015-04-29  7:41 ` [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino Michal Suchanek
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Michal Suchanek @ 2015-04-29  8:21 UTC (permalink / raw)
  To: linux-arm-kernel


Hello,

this adds spi2 bus on A10s Olinuxino Micro board.

Thanks

Michal


Michal Suchanek (4):
  ARM: dts: sunxi: sun5i: add spi2_pins_b.
  ARM: dts: sunxi: sun5i-a10s: add spi2_pins_a.
  ARM: dts: sunxi: A10s Olinuxino Micro: add spi2.
  ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino.

 arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts | 9 +++++++++
 arch/arm/boot/dts/sun5i-a10s.dtsi                | 8 ++++++++
 arch/arm/boot/dts/sun5i.dtsi                     | 7 +++++++
 3 files changed, 24 insertions(+)

-- 
2.1.4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/4] ARM: dts: sunxi: sun5i: add spi2_pins_b.
  2015-04-29  8:00 ` [PATCH 1/4] ARM: dts: sunxi: sun5i: add spi2_pins_b Michal Suchanek
@ 2015-05-01 20:22   ` Maxime Ripard
  0 siblings, 0 replies; 7+ messages in thread
From: Maxime Ripard @ 2015-05-01 20:22 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Apr 29, 2015 at 10:00:27AM +0200, Michal Suchanek wrote:
> spi2_pins_a are only available on A10s

spi2_pins_a doesn't mean anything without any context.

Anyone should be able to read this commit log alone and understand
what it's about and why you are doing it.

It's clearly not the case here.

> 
> Signed-off-by: Michal Suchanek <hramrach@gmail.com>
> ---
>  arch/arm/boot/dts/sun5i.dtsi | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi
> index 186d25f..9f22c28 100644
> --- a/arch/arm/boot/dts/sun5i.dtsi
> +++ b/arch/arm/boot/dts/sun5i.dtsi
> @@ -551,6 +551,13 @@
>  				allwinner,drive = <SUN4I_PINCTRL_30_MA>;
>  				allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
>  			};
> +
> +			spi2_pins_b: spi2 at 1 {

You should probably name that group spi2_pins_a. It feels a bit weird
to have a spi2_pins_b group without its "a" counterpart.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150501/7496b708/attachment-0001.sig>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino.
  2015-04-29  7:41 ` [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino Michal Suchanek
@ 2015-05-01 20:24   ` Maxime Ripard
  0 siblings, 0 replies; 7+ messages in thread
From: Maxime Ripard @ 2015-05-01 20:24 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Apr 29, 2015 at 09:41:17AM +0200, Michal Suchanek wrote:
> SPI aliases give nicer spidev device node names.
> 
> Signed-off-by: Michal Suchanek <hramrach@gmail.com>
> ---
>  arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts b/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
> index cd8d08a..a49fc6a 100644
> --- a/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
> +++ b/arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts
> @@ -63,6 +63,9 @@
>  		serial0 = &uart0;
>  		serial1 = &uart2;
>  		serial2 = &uart3;
> +		spi0 = &spi0;
> +		spi1 = &spi1;
> +		spi2 = &spi2;

That patch should be merged with the patch 3, and since there's only
spi2 in use on that board, it should be aliased to spi0.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150501/48a47c75/attachment-0001.sig>

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-05-01 20:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-29  8:21 [PATCH 0/4] A10s Olinuxino Micro DT SPI patches Michal Suchanek
2015-04-29  7:41 ` [PATCH 4/4] ARM: dts: sunxi: Add SPI aliases on A10s Olinuxino Michal Suchanek
2015-05-01 20:24   ` Maxime Ripard
2015-04-29  7:59 ` [PATCH 3/4] ARM: dts: sunxi: A10s Olinuxino Micro: add spi2 Michal Suchanek
2015-04-29  8:00 ` [PATCH 1/4] ARM: dts: sunxi: sun5i: add spi2_pins_b Michal Suchanek
2015-05-01 20:22   ` Maxime Ripard
2015-04-29  8:02 ` [PATCH 2/4] ARM: dts: sunxi: sun5i-a10s: add spi2_pins_a Michal Suchanek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).