* [PATCH] ARM: mach-imx/mx31_3ds: Fix IOMUX for SPI1 signals
@ 2011-06-16 16:42 Fabio Estevam
2011-06-20 7:26 ` Sascha Hauer
0 siblings, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2011-06-16 16:42 UTC (permalink / raw)
To: linux-arm-kernel
The LCD on the mx31_3ds board is connected to CSPI1.
Fix the IOMUX settings so that LCD can work.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
arch/arm/mach-imx/mach-mx31_3ds.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/mach-imx/mach-mx31_3ds.c b/arch/arm/mach-imx/mach-mx31_3ds.c
index ab2a626..a6160ee 100644
--- a/arch/arm/mach-imx/mach-mx31_3ds.c
+++ b/arch/arm/mach-imx/mach-mx31_3ds.c
@@ -53,11 +53,8 @@ static int mx31_3ds_pins[] = {
MX31_PIN_RXD1__RXD1,
IOMUX_MODE(MX31_PIN_GPIO1_1, IOMUX_CONFIG_GPIO),
/*SPI0*/
- MX31_PIN_CSPI1_SCLK__SCLK,
- MX31_PIN_CSPI1_MOSI__MOSI,
- MX31_PIN_CSPI1_MISO__MISO,
- MX31_PIN_CSPI1_SPI_RDY__SPI_RDY,
- MX31_PIN_CSPI1_SS2__SS2, /* CS for LCD */
+ IOMUX_MODE(MX31_PIN_DSR_DCE1, IOMUX_CONFIG_ALT1),
+ IOMUX_MODE(MX31_PIN_RI_DCE1, IOMUX_CONFIG_ALT1),
/* SPI 1 */
MX31_PIN_CSPI2_SCLK__SCLK,
MX31_PIN_CSPI2_MOSI__MOSI,
@@ -690,6 +687,9 @@ static void __init mx31_3ds_init(void)
int ret;
imx31_soc_init();
+
+ /* Configure SPI1 IOMUX */
+ mxc_iomux_set_gpr(MUX_PGP_CSPI_BB, true);
mxc_iomux_setup_multiple_pins(mx31_3ds_pins, ARRAY_SIZE(mx31_3ds_pins),
"mx31_3ds");
--
1.6.0.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] ARM: mach-imx/mx31_3ds: Fix IOMUX for SPI1 signals
2011-06-16 16:42 [PATCH] ARM: mach-imx/mx31_3ds: Fix IOMUX for SPI1 signals Fabio Estevam
@ 2011-06-20 7:26 ` Sascha Hauer
2011-06-20 12:43 ` Fabio Estevam
0 siblings, 1 reply; 4+ messages in thread
From: Sascha Hauer @ 2011-06-20 7:26 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 16, 2011 at 01:42:52PM -0300, Fabio Estevam wrote:
> The LCD on the mx31_3ds board is connected to CSPI1.
>From bogus@does.not.exist.com Wed Jun 1 12:03:18 2011
From: bogus@does.not.exist.com ()
Date: Wed, 01 Jun 2011 16:03:18 -0000
Subject: No subject
Message-ID: <mailman.29.1308554824.24103.linux-arm-kernel@lists.infradead.org>
instead of adding it. Can you explain a bit more?
Sascha
>
> Fix the IOMUX settings so that LCD can work.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> arch/arm/mach-imx/mach-mx31_3ds.c | 10 +++++-----
> 1 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/mach-imx/mach-mx31_3ds.c b/arch/arm/mach-imx/mach-mx31_3ds.c
> index ab2a626..a6160ee 100644
> --- a/arch/arm/mach-imx/mach-mx31_3ds.c
> +++ b/arch/arm/mach-imx/mach-mx31_3ds.c
> @@ -53,11 +53,8 @@ static int mx31_3ds_pins[] = {
> MX31_PIN_RXD1__RXD1,
> IOMUX_MODE(MX31_PIN_GPIO1_1, IOMUX_CONFIG_GPIO),
> /*SPI0*/
> - MX31_PIN_CSPI1_SCLK__SCLK,
> - MX31_PIN_CSPI1_MOSI__MOSI,
> - MX31_PIN_CSPI1_MISO__MISO,
> - MX31_PIN_CSPI1_SPI_RDY__SPI_RDY,
> - MX31_PIN_CSPI1_SS2__SS2, /* CS for LCD */
> + IOMUX_MODE(MX31_PIN_DSR_DCE1, IOMUX_CONFIG_ALT1),
> + IOMUX_MODE(MX31_PIN_RI_DCE1, IOMUX_CONFIG_ALT1),
> /* SPI 1 */
> MX31_PIN_CSPI2_SCLK__SCLK,
> MX31_PIN_CSPI2_MOSI__MOSI,
> @@ -690,6 +687,9 @@ static void __init mx31_3ds_init(void)
> int ret;
>
> imx31_soc_init();
> +
> + /* Configure SPI1 IOMUX */
> + mxc_iomux_set_gpr(MUX_PGP_CSPI_BB, true);
>
> mxc_iomux_setup_multiple_pins(mx31_3ds_pins, ARRAY_SIZE(mx31_3ds_pins),
> "mx31_3ds");
> --
> 1.6.0.4
>
>
>
--
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 |
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH] ARM: mach-imx/mx31_3ds: Fix IOMUX for SPI1 signals
2011-06-20 7:26 ` Sascha Hauer
@ 2011-06-20 12:43 ` Fabio Estevam
2011-06-20 13:10 ` Sascha Hauer
0 siblings, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2011-06-20 12:43 UTC (permalink / raw)
To: linux-arm-kernel
Hi Sascha,
On Mon, Jun 20, 2011 at 4:26 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> On Thu, Jun 16, 2011 at 01:42:52PM -0300, Fabio Estevam wrote:
>> The LCD on the mx31_3ds board is connected to CSPI1.
>
> From the description I would say you remove exactly the setup we want
> instead of adding it. Can you explain a bit more?
Sure, the original code was assuming that the CSPI1 pins on the
MX31PDK were the primary pin function, which is incorrect.
On MX31PDK board these are the pins that provide CSPI1 functionality:
DSR_DCE1 (ALT mode 1) --> CSPI1_CLK
RI_DCE1 (ALT mode 1) --> CSPI1_RDY
DTR_DTE1 -->CSI1_MOSI
DSR_DTE1 --> CSPI1_MISO
DTR_DCE2 ---> CSPI1_SS2
The 3 IOMUX settings above are done via GPR as per Table A-1 of the MX31RM.
So the patch I sent fixes the CSPI1 IOMUX and makes the LCD to be functional.
I think we haven't seen this before because probably Redboot was doing
the IOMUX for us. After I switched to U-boot I was not able to get the
LCD to work and now with this patch it is working fine. FSL BSP also
does the IOMUX in the same way.
If you prefer I can post a v2 of this patch with a better explanation
on the commit message.
Regards,
Fabio Estevam
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: mach-imx/mx31_3ds: Fix IOMUX for SPI1 signals
2011-06-20 12:43 ` Fabio Estevam
@ 2011-06-20 13:10 ` Sascha Hauer
0 siblings, 0 replies; 4+ messages in thread
From: Sascha Hauer @ 2011-06-20 13:10 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jun 20, 2011 at 09:43:13AM -0300, Fabio Estevam wrote:
> Hi Sascha,
>
> On Mon, Jun 20, 2011 at 4:26 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> > On Thu, Jun 16, 2011 at 01:42:52PM -0300, Fabio Estevam wrote:
> >> The LCD on the mx31_3ds board is connected to CSPI1.
> >
> > From the description I would say you remove exactly the setup we want
> > instead of adding it. Can you explain a bit more?
>
> Sure, the original code was assuming that the CSPI1 pins on the
> MX31PDK were the primary pin function, which is incorrect.
>
> On MX31PDK board these are the pins that provide CSPI1 functionality:
>
> DSR_DCE1 (ALT mode 1) --> CSPI1_CLK
> RI_DCE1 (ALT mode 1) --> CSPI1_RDY
>
> DTR_DTE1 -->CSI1_MOSI
> DSR_DTE1 --> CSPI1_MISO
> DTR_DCE2 ---> CSPI1_SS2
>
> The 3 IOMUX settings above are done via GPR as per Table A-1 of the MX31RM.
>
> So the patch I sent fixes the CSPI1 IOMUX and makes the LCD to be functional.
>
> I think we haven't seen this before because probably Redboot was doing
> the IOMUX for us. After I switched to U-boot I was not able to get the
> LCD to work and now with this patch it is working fine. FSL BSP also
> does the IOMUX in the same way.
>
> If you prefer I can post a v2 of this patch with a better explanation
> on the commit message.
>
Yes please.
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 |
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-06-20 13:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-16 16:42 [PATCH] ARM: mach-imx/mx31_3ds: Fix IOMUX for SPI1 signals Fabio Estevam
2011-06-20 7:26 ` Sascha Hauer
2011-06-20 12:43 ` Fabio Estevam
2011-06-20 13:10 ` Sascha Hauer
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).