diff mbox

[PATCHv2,2/3] ARM: dts: mxs Add additionnal muxing options for SSP3

Message ID 1346666749-6094-2-git-send-email-maxime.ripard@free-electrons.com (mailing list archive)
State New, archived
Headers show

Commit Message

Maxime Ripard Sept. 3, 2012, 10:05 a.m. UTC
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
---
 arch/arm/boot/dts/imx28.dtsi |   11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Shawn Guo Sept. 3, 2012, 2:53 p.m. UTC | #1
On Mon, Sep 03, 2012 at 12:05:48PM +0200, Maxime Ripard wrote:
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
>  arch/arm/boot/dts/imx28.dtsi |   11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx28.dtsi b/arch/arm/boot/dts/imx28.dtsi
> index 8497a0e..6c51ff4 100644
> --- a/arch/arm/boot/dts/imx28.dtsi
> +++ b/arch/arm/boot/dts/imx28.dtsi
> @@ -586,6 +586,17 @@
>  					fsl,voltage = <1>;
>  					fsl,pull-up = <1>;
>  				};
> +
> +				spi3_pins_b: spi3@1 {
> +					reg = <1>;
> +					fsl,pinmux-ids = <
> +						0x0181 /* MX28_PAD_GPMI_RDN__SSP3_SCK */
> +						0x01c1 /* MX28_PAD_GPMI_RESETN__SSP3_CMD */

The spi only needs two pins to work?

Has spi3_pins_a been defined?  I haven't seen it.

Regards,
Shawn

> +					>;
> +					fsl,drive-strength = <1>;
> +					fsl,voltage = <1>;
> +					fsl,pull-up = <1>;
> +				};
>  			};
>  
>  			digctl@8001c000 {
> -- 
> 1.7.9.5
>
Maxime Ripard Sept. 3, 2012, 3:22 p.m. UTC | #2
Le 03/09/2012 16:53, Shawn Guo a écrit :
> On Mon, Sep 03, 2012 at 12:05:48PM +0200, Maxime Ripard wrote:
>> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
>> ---
>>  arch/arm/boot/dts/imx28.dtsi |   11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/imx28.dtsi b/arch/arm/boot/dts/imx28.dtsi
>> index 8497a0e..6c51ff4 100644
>> --- a/arch/arm/boot/dts/imx28.dtsi
>> +++ b/arch/arm/boot/dts/imx28.dtsi
>> @@ -586,6 +586,17 @@
>>  					fsl,voltage = <1>;
>>  					fsl,pull-up = <1>;
>>  				};
>> +
>> +				spi3_pins_b: spi3@1 {
>> +					reg = <1>;
>> +					fsl,pinmux-ids = <
>> +						0x0181 /* MX28_PAD_GPMI_RDN__SSP3_SCK */
>> +						0x01c1 /* MX28_PAD_GPMI_RESETN__SSP3_CMD */
> 
> The spi only needs two pins to work?

Oops. I only used MOSI, so I forgot to mux MISO.
Sorry about that, this will be fixed in the new version.

> Has spi3_pins_a been defined?  I haven't seen it.

It's the same thing as the first patch. I had Marek's last patch in my
tree that adds spi3_pins_a. I will change it in the next version.

Thanks,
Maxime
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx28.dtsi b/arch/arm/boot/dts/imx28.dtsi
index 8497a0e..6c51ff4 100644
--- a/arch/arm/boot/dts/imx28.dtsi
+++ b/arch/arm/boot/dts/imx28.dtsi
@@ -586,6 +586,17 @@ 
 					fsl,voltage = <1>;
 					fsl,pull-up = <1>;
 				};
+
+				spi3_pins_b: spi3@1 {
+					reg = <1>;
+					fsl,pinmux-ids = <
+						0x0181 /* MX28_PAD_GPMI_RDN__SSP3_SCK */
+						0x01c1 /* MX28_PAD_GPMI_RESETN__SSP3_CMD */
+					>;
+					fsl,drive-strength = <1>;
+					fsl,voltage = <1>;
+					fsl,pull-up = <1>;
+				};
 			};
 
 			digctl@8001c000 {