[PATCH 1/3] ARM: dts: Add aliases for i.MX27 SPI controller

[PATCH 1/3] ARM: dts: Add aliases for i.MX27 SPI controller

[Alexander Shiyan]

Add aliases to determine the proper SPI bus number.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 arch/arm/boot/dts/imx27.dtsi | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
index ff4bd48..cd85828 100644
--- a/arch/arm/boot/dts/imx27.dtsi
+++ b/arch/arm/boot/dts/imx27.dtsi
@@ -25,6 +25,9 @@
 		gpio3 = &gpio4;
 		gpio4 = &gpio5;
 		gpio5 = &gpio6;
+		spi0 = &cspi1;
+		spi1 = &cspi2;
+		spi2 = &cspi3;
 	};
 
 	avic: avic-interrupt-controller at e0000000 {
-- 
1.8.1.5

[PATCH 2/3] ARM: dts: Add SPI support for i.MX27 Phytec PCM038 module

[Alexander Shiyan]

Added SPI node to imx27-phytec-phycore DT file.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 arch/arm/boot/dts/imx27-phytec-phycore.dts | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/imx27-phytec-phycore.dts b/arch/arm/boot/dts/imx27-phytec-phycore.dts
index fe64e3a..b0e9372 100644
--- a/arch/arm/boot/dts/imx27-phytec-phycore.dts
+++ b/arch/arm/boot/dts/imx27-phytec-phycore.dts
@@ -72,6 +72,12 @@
 	};
 };
 
+&cspi1 {
+	fsl,spi-num-chipselects = <1>;
+	cs-gpios = <&gpio4 28 0>;
+	status = "okay";
+};
+
 &nfc {
 	nand-bus-width = <8>;
 	nand-ecc-mode = "hw";
-- 
1.8.1.5

[PATCH 3/3] ARM: dts: Add MC13783 support for i.MX27 Phytec PCM038 module

[Alexander Shiyan]

Added MC13783 node to imx27-phytec-phycore DT file.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 arch/arm/boot/dts/imx27-phytec-phycore.dts | 92 ++++++++++++++++++++++++++++++
 1 file changed, 92 insertions(+)

diff --git a/arch/arm/boot/dts/imx27-phytec-phycore.dts b/arch/arm/boot/dts/imx27-phytec-phycore.dts
index b0e9372..5c4fea9 100644
--- a/arch/arm/boot/dts/imx27-phytec-phycore.dts
+++ b/arch/arm/boot/dts/imx27-phytec-phycore.dts
@@ -76,6 +76,98 @@
 	fsl,spi-num-chipselects = <1>;
 	cs-gpios = <&gpio4 28 0>;
 	status = "okay";
+
+	pmic: mc13783 at 0 {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		compatible = "fsl,mc13783";
+		spi-max-frequency = <20000000>;
+		reg = <0>;
+		interrupt-parent = <&gpio2>;
+		interrupts = <23 0x4>;
+		fsl,mc13xxx-uses-adc;
+
+		regulators {
+			sw1a_reg: sw1a {
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <1200000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			sw1b_reg: sw1b {
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <1200000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			sw2a_reg: sw2a {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			sw2b_reg: sw2b {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			sw3_reg: sw3 {
+				regulator-min-microvolt = <5000000>;
+				regulator-max-microvolt = <5000000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			vaudio_reg: vaudio {
+				regulator-min-microvolt = <2775000>;
+				regulator-max-microvolt = <2775000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			violo_reg: violo {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			viohi_reg: viohi {
+				regulator-min-microvolt = <2775000>;
+				regulator-max-microvolt = <2775000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			vgen_reg: vgen {
+				regulator-min-microvolt = <1500000>;
+				regulator-max-microvolt = <1500000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			vcam_reg: vcam {
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
+			};
+			vrf1_reg: vrf1 {
+				regulator-min-microvolt = <2775000>;
+				regulator-max-microvolt = <2775000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			vrf2_reg: vrf2 {
+				regulator-min-microvolt = <2775000>;
+				regulator-max-microvolt = <2775000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+			vmmc1_reg: vmmc1 {
+				regulator-min-microvolt = <1600000>;
+				regulator-max-microvolt = <3000000>;
+			};
+			gpo1_reg: gpo1 {
+			};
+			pwgt1spi_reg: pwgt1spi {
+			};
+		};
+	};
 };
 
 &nfc {
-- 
1.8.1.5

[PATCH 3/3] ARM: dts: Add MC13783 support for i.MX27 Phytec PCM038 module

[Sascha Hauer]

On Wed, May 01, 2013 at 02:46:59PM +0400, Alexander Shiyan wrote:
> Added MC13783 node to imx27-phytec-phycore DT file.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
>  arch/arm/boot/dts/imx27-phytec-phycore.dts | 92 ++++++++++++++++++++++++++++++
>  1 file changed, 92 insertions(+)

You can merge this with the previous patch.

> 
> diff --git a/arch/arm/boot/dts/imx27-phytec-phycore.dts b/arch/arm/boot/dts/imx27-phytec-phycore.dts
> index b0e9372..5c4fea9 100644
> --- a/arch/arm/boot/dts/imx27-phytec-phycore.dts
> +++ b/arch/arm/boot/dts/imx27-phytec-phycore.dts
> @@ -76,6 +76,98 @@
>  	fsl,spi-num-chipselects = <1>;
>  	cs-gpios = <&gpio4 28 0>;
>  	status = "okay";
> +
> +	pmic: mc13783 at 0 {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		compatible = "fsl,mc13783";
> +		spi-max-frequency = <20000000>;
> +		reg = <0>;
> +		interrupt-parent = <&gpio2>;
> +		interrupts = <23 0x4>;
> +		fsl,mc13xxx-uses-adc;
> +
> +		regulators {
> +			sw1a_reg: sw1a {
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <1200000>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +			sw1b_reg: sw1b {

Please add a blank line after each node.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Re[2]: [PATCH 3/3] ARM: dts: Add MC13783 support for i.MX27 Phytec PCM038 module

[Alexander Shiyan]

> On Wed, May 01, 2013 at 02:46:59PM +0400, Alexander Shiyan wrote:
> > Added MC13783 node to imx27-phytec-phycore DT file.
> > 
> > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > ---
> >  arch/arm/boot/dts/imx27-phytec-phycore.dts | 92 ++++++++++++++++++++++++++++++
> >  1 file changed, 92 insertions(+)
> 
> You can merge this with the previous patch.

Part 2 just adds a SPI host for the target. Is this merge is really necessary?

---

[PATCH 3/3] ARM: dts: Add MC13783 support for i.MX27 Phytec PCM038 module

[Sascha Hauer]

On Wed, May 01, 2013 at 07:40:37PM +0400, Alexander Shiyan wrote:
> > On Wed, May 01, 2013 at 02:46:59PM +0400, Alexander Shiyan wrote:
> > > Added MC13783 node to imx27-phytec-phycore DT file.
> > > 
> > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > > ---
> > >  arch/arm/boot/dts/imx27-phytec-phycore.dts | 92 ++++++++++++++++++++++++++++++
> > >  1 file changed, 92 insertions(+)
> > 
> > You can merge this with the previous patch.
> 
> Part 2 just adds a SPI host for the target. Is this merge is really necessary?

The SPI host does nothing without devices registered on it, so why not
make 2/3 and 3/3 a single patch?

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Re[2]: [PATCH 3/3] ARM: dts: Add MC13783 support for i.MX27 Phytec PCM038 module

[Alexander Shiyan]

> On Wed, May 01, 2013 at 07:40:37PM +0400, Alexander Shiyan wrote:
> > > On Wed, May 01, 2013 at 02:46:59PM +0400, Alexander Shiyan wrote:
> > > > Added MC13783 node to imx27-phytec-phycore DT file.
> > > > 
> > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > > > ---
> > > >  arch/arm/boot/dts/imx27-phytec-phycore.dts | 92 ++++++++++++++++++++++++++++++
> > > >  1 file changed, 92 insertions(+)
> > > 
> > > You can merge this with the previous patch.
> > 
> > Part 2 just adds a SPI host for the target. Is this merge is really necessary?
> 
> The SPI host does nothing without devices registered on it, so why not
> make 2/3 and 3/3 a single patch?

At point of 2/3 user can use SPI0.0 as spidev. And in th

---

Re[3]: [PATCH 3/3] ARM: dts: Add MC13783 support for i.MX27 Phytec PCM038 module

[Alexander Shiyan]

> > On Wed, May 01, 2013 at 07:40:37PM +0400, Alexander Shiyan wrote:
> > > > On Wed, May 01, 2013 at 02:46:59PM +0400, Alexander Shiyan wrote:
> > > > > Added MC13783 node to imx27-phytec-phycore DT file.
> > > > > 
> > > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > > > > ---
> > > > >  arch/arm/boot/dts/imx27-phytec-phycore.dts | 92 ++++++++++++++++++++++++++++++
> > > > >  1 file changed, 92 insertions(+)
> > > > 
> > > > You can merge this with the previous patch.
> > > 
> > > Part 2 just adds a SPI host for the target. Is this merge is really necessary?
> > 
> > The SPI host does nothing without devices registered on it, so why not
> > make 2/3 and 3/3 a single patch?
> 
> At point of 2/3 user can use SPI0.0 as spidev. And in th

Please ignore this message.
Thanks.

---

[PATCH 1/3] ARM: dts: Add aliases for i.MX27 SPI controller

[Shawn Guo]

On Wed, May 01, 2013 at 02:46:57PM +0400, Alexander Shiyan wrote:
> Add aliases to determine the proper SPI bus number.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>

Applied, thanks.

> ---
>  arch/arm/boot/dts/imx27.dtsi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index ff4bd48..cd85828 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -25,6 +25,9 @@
>  		gpio3 = &gpio4;
>  		gpio4 = &gpio5;
>  		gpio5 = &gpio6;
> +		spi0 = &cspi1;
> +		spi1 = &cspi2;
> +		spi2 = &cspi3;
>  	};
>  
>  	avic: avic-interrupt-controller at e0000000 {
> -- 
> 1.8.1.5
>