[PATCH 33/40] ARM: mx5/iomux-mx51: Fix input path of some pins in gpio mode

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Mon Sep 27 10:47:51 EDT 2010


Hi Lothar,

On Mon, Sep 27, 2010 at 04:31:34PM +0200, Sascha Hauer wrote:
> On Mon, Sep 27, 2010 at 04:16:51PM +0200, Lothar Waßmann wrote:
> > Hi,
> > 
> > > diff --git a/arch/arm/plat-mxc/include/mach/iomux-mx51.h b/arch/arm/plat-mxc/include/mach/iomux-mx51.h
> > > index 9250273..639d631 100644
> > > --- a/arch/arm/plat-mxc/include/mach/iomux-mx51.h
> > > +++ b/arch/arm/plat-mxc/include/mach/iomux-mx51.h
> > > @@ -274,15 +274,15 @@ typedef enum iomux_config {
> > >  #define MX51_PAD_USBH1_DATA5__USBH1_DATA5       IOMUX_PAD(0x69C, 0x29C, 0, 0x0,   0, MX51_USBH1_PAD_CTRL)
> > >  #define MX51_PAD_USBH1_DATA6__USBH1_DATA6       IOMUX_PAD(0x6A0, 0x2A0, 0, 0x0,   0, MX51_USBH1_PAD_CTRL)
> > >  #define MX51_PAD_USBH1_DATA7__USBH1_DATA7       IOMUX_PAD(0x6A4, 0x2A4, 0, 0x0,   0, MX51_USBH1_PAD_CTRL)
> > > -#define MX51_PAD_DI1_PIN11__GPIO_3_0            IOMUX_PAD(0x6A8, 0x2A8, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DI1_PIN12__GPIO_3_1            IOMUX_PAD(0x6AC, 0x2AC, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DI1_PIN13__GPIO_3_2            IOMUX_PAD(0x6B0, 0x2B0, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DI1_D0_CS__GPIO_3_3            IOMUX_PAD(0x6B4, 0x2B4, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DI1_D1_CS__GPIO_3_4            IOMUX_PAD(0x6B8, 0x2B8, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DISPB2_SER_DIN__GPIO_3_5       IOMUX_PAD(0x6BC, 0x2BC, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DISPB2_SER_DIO__GPIO_3_6       IOMUX_PAD(0x6C0, 0x2C0, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DISPB2_SER_CLK__GPIO_3_7       IOMUX_PAD(0x6C4, 0x2C4, 4, 0x0,   0, NO_PAD_CTRL)
> > > -#define MX51_PAD_DISPB2_SER_RS__GPIO_3_8        IOMUX_PAD(0x6C8, 0x2C8, 4, 0x0,   0, NO_PAD_CTRL)
> > > +#define MX51_PAD_DI1_PIN11__GPIO_3_0            IOMUX_PAD(0x6A8, 0x2A8, 4, 0x974, 1, NO_PAD_CTRL)
> >                                                                               ^^^^^
> > Where did you get this from?
> > According to my copy of the i.MX51 Reference Manual (MCIMX51RM Rev. 0 11/2009)
> > the register at 0x974 is IOMUXC_FEC_FEC_TX_CLK_SELECT_INPUT which
> > controls the pads:
> > |0: Selecting BGA contact: NANDF_RDY_INT for Mode: ALT1.
> > |1: Selecting BGA contact: DISP2_DAT13 for Mode: ALT2.
> > but not DI1_PIN11. DI1_PIN11 does not have any input mux.
> 
> Indeed, checked in the newer rev1 Manual from 2/2010 aswell.
So I assume the definition for MX51_PAD_DI1_PIN11__GPIO_3_0 has to stay
as it was, i.e.

#define MX51_PAD_DI1_PIN11__GPIO_3_0 IOMUX_PAD(0x6A8, 0x2A8, 4, 0x0, 0, NO_PAD_CTRL)

Will fix up accordingly.

@Lothar: good catch, I'm glad you're that attentive.

Thanks
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list