* [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch>
@ 2012-07-04 12:10 Attila Kinali
2012-07-04 12:50 ` Fabio Estevam
2012-07-04 13:53 ` Shawn Guo
0 siblings, 2 replies; 6+ messages in thread
From: Attila Kinali @ 2012-07-04 12:10 UTC (permalink / raw)
To: linux-arm-kernel
---
arch/arm/boot/dts/imx23.dtsi | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi
index 8c5f999..df0ae8d 100644
--- a/arch/arm/boot/dts/imx23.dtsi
+++ b/arch/arm/boot/dts/imx23.dtsi
@@ -134,6 +134,21 @@
fsl,pinmux-ids = <0x2010 0x2060>;
fsl,pull-up = <0>;
};
+
+ mmc1_8bit_pins_a: mmc1-8bit at 0 {
+ reg = <0>;
+ fsl,pinmux-ids = <0x0002 0x0012 0x0022
+ 0x0032 0x0042 0x0052 0x0062
+ 0x0072 0x0142 0x0132 0x0182>;
+ fsl,drive-strength = <1>;
+ fsl,voltage = <1>;
+ fsl,pull-up = <1>;
+ };
+
+ mmc1_pins_fixup: mmc1-pins-fixup {
+ fsl,pinmux-ids = <0x0132 0x0182>;
+ fsl,pull-up = <0>;
+ };
};
digctl at 8001c000 {
--
1.7.10
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch>
2012-07-04 12:10 [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch> Attila Kinali
@ 2012-07-04 12:50 ` Fabio Estevam
2012-07-04 13:53 ` Shawn Guo
1 sibling, 0 replies; 6+ messages in thread
From: Fabio Estevam @ 2012-07-04 12:50 UTC (permalink / raw)
To: linux-arm-kernel
Hi Attila,
On Wed, Jul 4, 2012 at 9:10 AM, Attila Kinali <attila@kinali.ch> wrote:
> ---
> arch/arm/boot/dts/imx23.dtsi | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
Your Signed-off-by line appears in the Subject. Please fix it and resend.
Regards,
Fabio Estevam
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch>
2012-07-04 12:10 [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch> Attila Kinali
2012-07-04 12:50 ` Fabio Estevam
@ 2012-07-04 13:53 ` Shawn Guo
2012-07-04 15:09 ` Attila Kinali
1 sibling, 1 reply; 6+ messages in thread
From: Shawn Guo @ 2012-07-04 13:53 UTC (permalink / raw)
To: linux-arm-kernel
Suggested subject prefix "ARM: dts: imx23: ..."
On Wed, Jul 04, 2012 at 02:10:33PM +0200, Attila Kinali wrote:
> ---
> arch/arm/boot/dts/imx23.dtsi | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi
> index 8c5f999..df0ae8d 100644
> --- a/arch/arm/boot/dts/imx23.dtsi
> +++ b/arch/arm/boot/dts/imx23.dtsi
> @@ -134,6 +134,21 @@
> fsl,pinmux-ids = <0x2010 0x2060>;
> fsl,pull-up = <0>;
> };
> +
> + mmc1_8bit_pins_a: mmc1-8bit at 0 {
> + reg = <0>;
> + fsl,pinmux-ids = <0x0002 0x0012 0x0022
> + 0x0032 0x0042 0x0052 0x0062
> + 0x0072 0x0142 0x0132 0x0182>;
Please take a look at the following branch to see how these should be
formated now.
http://git.linaro.org/gitweb?p=people/shawnguo/linux-2.6.git;a=shortlog;h=refs/heads/mxs/dt-for-3.6
> + fsl,drive-strength = <1>;
> + fsl,voltage = <1>;
> + fsl,pull-up = <1>;
> + };
> +
> + mmc1_pins_fixup: mmc1-pins-fixup {
> + fsl,pinmux-ids = <0x0132 0x0182>;
> + fsl,pull-up = <0>;
> + };
I do not want to have these pinctrl setup added along. Please submit
the changes together with the <board>.dts which uses them.
Regards,
Shawn
> };
>
> digctl at 8001c000 {
> --
> 1.7.10
>
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch>
2012-07-04 13:53 ` Shawn Guo
@ 2012-07-04 15:09 ` Attila Kinali
2012-07-04 15:26 ` Shawn Guo
0 siblings, 1 reply; 6+ messages in thread
From: Attila Kinali @ 2012-07-04 15:09 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, 4 Jul 2012 21:53:38 +0800
Shawn Guo <shawn.guo@linaro.org> wrote:
> Suggested subject prefix "ARM: dts: imx23: ..."
Ok
> > + fsl,pinmux-ids = <0x0002 0x0012 0x0022
> > + 0x0032 0x0042 0x0052 0x0062
> > + 0x0072 0x0142 0x0132 0x0182>;
>
> Please take a look at the following branch to see how these should be
> formated now.
>
> http://git.linaro.org/gitweb?p=people/shawnguo/linux-2.6.git;a=shortlog;h=refs/heads/mxs/dt-for-3.6
Ok
> > + mmc1_pins_fixup: mmc1-pins-fixup {
> > + fsl,pinmux-ids = <0x0132 0x0182>;
> > + fsl,pull-up = <0>;
> > + };
>
> I do not want to have these pinctrl setup added along. Please submit
> the changes together with the <board>.dts which uses them.
I copied these from the mmc0 definition above. It could be these are
spillovers from the imx23-evk definition. Shall i move teh mmc0_pins_fixup
into the imx23-evk.dts file?
Attila Kinali
--
The trouble with you, Shev, is you don't say anything until you've saved
up a whole truckload of damned heavy brick arguments and then you dump
them all out and never look at the bleeding body mangled beneath the heap
-- Tirin, The Dispossessed, U. Le Guin
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch>
2012-07-04 15:09 ` Attila Kinali
@ 2012-07-04 15:26 ` Shawn Guo
2012-07-04 15:29 ` Attila Kinali
0 siblings, 1 reply; 6+ messages in thread
From: Shawn Guo @ 2012-07-04 15:26 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jul 04, 2012 at 05:09:24PM +0200, Attila Kinali wrote:
> > I do not want to have these pinctrl setup added along. Please submit
Sorry. s/along/alone
> > the changes together with the <board>.dts which uses them.
>
> I copied these from the mmc0 definition above. It could be these are
> spillovers from the imx23-evk definition. Shall i move teh mmc0_pins_fixup
> into the imx23-evk.dts file?
>
The changes in the patch look good. What I meant is we need have the
user of mmc1_8bit_pins_a and mmc1_pins_fixup added together. The
"user" means a <board>.dts which refers to these pins.
--
Regards,
Shawn
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch>
2012-07-04 15:26 ` Shawn Guo
@ 2012-07-04 15:29 ` Attila Kinali
0 siblings, 0 replies; 6+ messages in thread
From: Attila Kinali @ 2012-07-04 15:29 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, 4 Jul 2012 23:26:06 +0800
Shawn Guo <shawn.guo@linaro.org> wrote:
> On Wed, Jul 04, 2012 at 05:09:24PM +0200, Attila Kinali wrote:
> > > the changes together with the <board>.dts which uses them.
> >
> > I copied these from the mmc0 definition above. It could be these are
> > spillovers from the imx23-evk definition. Shall i move teh mmc0_pins_fixup
> > into the imx23-evk.dts file?
> >
> The changes in the patch look good. What I meant is we need have the
> user of mmc1_8bit_pins_a and mmc1_pins_fixup added together. The
> "user" means a <board>.dts which refers to these pins.
Ah.. ok.. Then it will have to wait until i have the system completely
up and running. I'm still in the bring up phase and there will be
still changes in the <board>.dts (not to mention that i didn't know
about dts until yesterday and am likely to have bugs in the whole thing)
Attila Kinali
--
The trouble with you, Shev, is you don't say anything until you've saved
up a whole truckload of damned heavy brick arguments and then you dump
them all out and never look at the bleeding body mangled beneath the heap
-- Tirin, The Dispossessed, U. Le Guin
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-07-04 15:29 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-04 12:10 [PATCH] i.mx23: add dts node for mmc1/ssp1 Signed-off-by: Attila Kinali <attila@kinali.ch> Attila Kinali
2012-07-04 12:50 ` Fabio Estevam
2012-07-04 13:53 ` Shawn Guo
2012-07-04 15:09 ` Attila Kinali
2012-07-04 15:26 ` Shawn Guo
2012-07-04 15:29 ` Attila Kinali
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).