[RFC][Patch V1] OMAP3: Mux Changes.

[RFC][Patch V1] OMAP3: Mux Changes.

[Pais, Allen]

Please ignore my previous mail.

Muxes for OMAP 3630.

Signed-off-by: Allen Pais <allen.pais@ti.com> diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c index b5fac32..93abb74 100644
--- a/arch/arm/mach-omap2/mux.c
+++ b/arch/arm/mach-omap2/mux.c
@@ -551,6 +551,42 @@ MUX_CFG_34XX("AF13_3430_MMC3_DAT3", 0x5e2,  MUX_CFG_34XX("AF26_34XX_SYS_NIRQ", 0x1E0,
 		OMAP3_WAKEUP_EN | OMAP34XX_PIN_INPUT_PULLUP |
 		OMAP34XX_MUX_MODE0)
+
+/*Muxes for 3630 */
+MUX_CFG_34XX("H26_3630_DSS_DATA18", 0x100,
+		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("H25_3630_DSS_DATA19", 0x102,
+		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("E28_3630_DSS_DATA20", 0x104,
+		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("J26_3630_DSS_DATA21", 0x106,
+		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AC27_3630_DSS_DATA22", 0x108,
+		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AC28_3630_DSS_DATA23", 0x10A,
+		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
+
+MUX_CFG_34XX("AF9_3630_ETKD8", 0x5EC,
+		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AG9_3630_ETKD9", 0x5EE,
+		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AG7_3630_ETKD12", 0x5F0,
+		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
+
+MUX_CFG_34XX("AA25_3630_UART2_TX", 0x178,
+		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AD25_3630_UART2_RX", 0x17A,
+		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AB25_3630_UART2_RTS", 0x176,
+		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("AB26_3630_UART2_CTS", 0x174,
+		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
+
+MUX_CFG_34XX("H20_UART3_RX_IRRX", 0x19E,
+		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN) 
+MUX_CFG_34XX("H21_UART3_TX_IRTX", 0x1A0,
+		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
+
 };
 
 #define OMAP34XX_PINS_SZ	ARRAY_SIZE(omap34xx_pins)
diff --git a/arch/arm/plat-omap/include/mach/mux.h b/arch/arm/plat-omap/include/mach/mux.h
index 0f49d2d..8d8cbe1 100644
--- a/arch/arm/plat-omap/include/mach/mux.h
+++ b/arch/arm/plat-omap/include/mach/mux.h
@@ -890,6 +890,32 @@ enum omap34xx_index {
 
 	/* SYS_NIRQ T2 INT1 */
 	AF26_34XX_SYS_NIRQ,
+
+	/*Muxes for 3630*/
+	K28_3630_CAM_D6,
+	L28_3630_CAM_D7,
+	K27_3630_CAM_D8,
+	L27_3630_CAM_D9,
+
+	H26_3630_DSS_DATA18,
+	H25_3630_DSS_DATA19,
+	E28_3630_DSS_DATA20,
+	J26_3630_DSS_DATA21,
+	AC27_3630_DSS_DATA22,
+	AC28_3630_DSS_DATA23,
+
+	AF9_3630_ETKD8,
+	AG9_3630_ETKD9,
+	AG7_3630_ETK12,
+
+	AA25_3630_UART2_TX,
+	AD25_3630_UART2_RX,
+	AB25_3630_UART2_RTS,
+	AB26_3630_UART2_CTS,
+
+	H20_UART3_RX_IRRX,
+	H21_UART3_TX_IRTX,
+
 };
 
 struct omap_mux_cfg {

RE: [RFC][Patch V1] OMAP3: Mux Changes.

[Menon, Nishanth]

Please send your patch using git send email. And generate your patch using git format patch

> -----Original Message-----
> From: Pais, Allen
> Sent: Thursday, October 15, 2009 4:38 AM
> 
> Please ignore my previous mail.
> 
> Muxes for OMAP 3630.
> 
> Signed-off-by: Allen Pais <allen.pais@ti.com> diff --git a/arch/arm/mach-
> omap2/mux.c b/arch/arm/mach-omap2/mux.c index b5fac32..93abb74 100644


> --- a/arch/arm/mach-omap2/mux.c
> +++ b/arch/arm/mach-omap2/mux.c
> @@ -551,6 +551,42 @@ MUX_CFG_34XX("AF13_3430_MMC3_DAT3", 0x5e2,
> MUX_CFG_34XX("AF26_34XX_SYS_NIRQ", 0x1E0,
>  		OMAP3_WAKEUP_EN | OMAP34XX_PIN_INPUT_PULLUP |
>  		OMAP34XX_MUX_MODE0)
> +
> +/*Muxes for 3630 */
> +MUX_CFG_34XX("H26_3630_DSS_DATA18", 0x100,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("H25_3630_DSS_DATA19", 0x102,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("E28_3630_DSS_DATA20", 0x104,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("J26_3630_DSS_DATA21", 0x106,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AC27_3630_DSS_DATA22", 0x108,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AC28_3630_DSS_DATA23", 0x10A,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
Why cant I do this in Board file?
> +
> +MUX_CFG_34XX("AF9_3630_ETKD8", 0x5EC,
> +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AG9_3630_ETKD9", 0x5EE,
> +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AG7_3630_ETKD12", 0x5F0,
> +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
Why am I doing this?
> +
> +MUX_CFG_34XX("AA25_3630_UART2_TX", 0x178,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AD25_3630_UART2_RX", 0x17A,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AB25_3630_UART2_RTS", 0x176,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("AB26_3630_UART2_CTS", 0x174,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
Please explain..
> +
> +MUX_CFG_34XX("H20_UART3_RX_IRRX", 0x19E,
> +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +MUX_CFG_34XX("H21_UART3_TX_IRTX", 0x1A0,
> +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +
>  };
??
> 
>  #define OMAP34XX_PINS_SZ	ARRAY_SIZE(omap34xx_pins)
> diff --git a/arch/arm/plat-omap/include/mach/mux.h b/arch/arm/plat-
> omap/include/mach/mux.h
> index 0f49d2d..8d8cbe1 100644
> --- a/arch/arm/plat-omap/include/mach/mux.h
> +++ b/arch/arm/plat-omap/include/mach/mux.h
> @@ -890,6 +890,32 @@ enum omap34xx_index {
> 
>  	/* SYS_NIRQ T2 INT1 */
>  	AF26_34XX_SYS_NIRQ,
> +
> +	/*Muxes for 3630*/
> +	K28_3630_CAM_D6,
> +	L28_3630_CAM_D7,
> +	K27_3630_CAM_D8,
> +	L27_3630_CAM_D9,
> +
> +	H26_3630_DSS_DATA18,
> +	H25_3630_DSS_DATA19,
> +	E28_3630_DSS_DATA20,
> +	J26_3630_DSS_DATA21,
> +	AC27_3630_DSS_DATA22,
> +	AC28_3630_DSS_DATA23,
> +
> +	AF9_3630_ETKD8,
> +	AG9_3630_ETKD9,
> +	AG7_3630_ETK12,
> +
> +	AA25_3630_UART2_TX,
> +	AD25_3630_UART2_RX,
> +	AB25_3630_UART2_RTS,
> +	AB26_3630_UART2_CTS,
> +
> +	H20_UART3_RX_IRRX,
> +	H21_UART3_TX_IRTX,
> +
>  };
> 
>  struct omap_mux_cfg {

RE: [RFC][Patch V1] OMAP3: Mux Changes.

[Menon, Nishanth]

Hi Allen,
a) A simple comment to all my comments: why cant we have these in bootloader and just simply leave the mux file alone?
b) Are you doing this for a specific platform or are they generic 3630 pin mux changes?

> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> owner@vger.kernel.org] On Behalf Of Menon, Nishanth
> Sent: Thursday, October 15, 2009 7:24 AM
> To: Pais, Allen; Aguirre Rodriguez, Sergio Alberto; linux-omap
> Cc: Chikkature Rajashekar, Madhusudhan; Pandita, Vikram; Gadiyar, Anand;
> Cousson, Benoit; Felipe Balbi; Kevin Hilman; Premi, Sanjeev; Shilimkar,
> Santosh; Tony Lindgren
> Subject: RE: [RFC][Patch V1] OMAP3: Mux Changes.
> 
> Please send your patch using git send email. And generate your patch using
> git format patch
> 
> > -----Original Message-----
> > From: Pais, Allen
> > Sent: Thursday, October 15, 2009 4:38 AM
> >
> > Please ignore my previous mail.
> >
> > Muxes for OMAP 3630.
> >
> > Signed-off-by: Allen Pais <allen.pais@ti.com> diff --git
> a/arch/arm/mach-
> > omap2/mux.c b/arch/arm/mach-omap2/mux.c index b5fac32..93abb74 100644
> 
> 
> > --- a/arch/arm/mach-omap2/mux.c
> > +++ b/arch/arm/mach-omap2/mux.c
> > @@ -551,6 +551,42 @@ MUX_CFG_34XX("AF13_3430_MMC3_DAT3", 0x5e2,
> > MUX_CFG_34XX("AF26_34XX_SYS_NIRQ", 0x1E0,
> >  		OMAP3_WAKEUP_EN | OMAP34XX_PIN_INPUT_PULLUP |
> >  		OMAP34XX_MUX_MODE0)
> > +
> > +/*Muxes for 3630 */
> > +MUX_CFG_34XX("H26_3630_DSS_DATA18", 0x100,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("H25_3630_DSS_DATA19", 0x102,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("E28_3630_DSS_DATA20", 0x104,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("J26_3630_DSS_DATA21", 0x106,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AC27_3630_DSS_DATA22", 0x108,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AC28_3630_DSS_DATA23", 0x10A,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> Why cant I do this in Board file?
I Really typed that??? Dumb me.. apologies on the noise.. I had meant bootloader..
> > +
> > +MUX_CFG_34XX("AF9_3630_ETKD8", 0x5EC,
> > +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AG9_3630_ETKD9", 0x5EE,
> > +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AG7_3630_ETKD12", 0x5F0,
> > +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> Why am I doing this?
> > +
> > +MUX_CFG_34XX("AA25_3630_UART2_TX", 0x178,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AD25_3630_UART2_RX", 0x17A,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AB25_3630_UART2_RTS", 0x176,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("AB26_3630_UART2_CTS", 0x174,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> Please explain..
> > +
> > +MUX_CFG_34XX("H20_UART3_RX_IRRX", 0x19E,
> > +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +MUX_CFG_34XX("H21_UART3_TX_IRTX", 0x1A0,
> > +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +
> >  };
> ??
> >
> >  #define OMAP34XX_PINS_SZ	ARRAY_SIZE(omap34xx_pins)
> > diff --git a/arch/arm/plat-omap/include/mach/mux.h b/arch/arm/plat-
> > omap/include/mach/mux.h
> > index 0f49d2d..8d8cbe1 100644
> > --- a/arch/arm/plat-omap/include/mach/mux.h
> > +++ b/arch/arm/plat-omap/include/mach/mux.h
> > @@ -890,6 +890,32 @@ enum omap34xx_index {
> >
> >  	/* SYS_NIRQ T2 INT1 */
> >  	AF26_34XX_SYS_NIRQ,
> > +
> > +	/*Muxes for 3630*/
> > +	K28_3630_CAM_D6,
> > +	L28_3630_CAM_D7,
> > +	K27_3630_CAM_D8,
> > +	L27_3630_CAM_D9,
> > +
> > +	H26_3630_DSS_DATA18,
> > +	H25_3630_DSS_DATA19,
> > +	E28_3630_DSS_DATA20,
> > +	J26_3630_DSS_DATA21,
> > +	AC27_3630_DSS_DATA22,
> > +	AC28_3630_DSS_DATA23,
> > +
> > +	AF9_3630_ETKD8,
> > +	AG9_3630_ETKD9,
> > +	AG7_3630_ETK12,
> > +
> > +	AA25_3630_UART2_TX,
> > +	AD25_3630_UART2_RX,
> > +	AB25_3630_UART2_RTS,
> > +	AB26_3630_UART2_CTS,
> > +
> > +	H20_UART3_RX_IRRX,
> > +	H21_UART3_TX_IRTX,
> > +
> >  };
> >
> >  struct omap_mux_cfg {
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


Regards,
Nishanth Menon

RE: [RFC][Patch V1] OMAP3: Mux Changes.

[Pais, Allen]

 


Hi Allen,
a) A simple comment to all my comments: why cant we have these in bootloader and just simply leave the mux file alone?
[Allen] Yes Nishanth, this would be a much cleaner approach. Even Santosh had suggested
The same, if we can conclude on a approach here, I can go ahead and do the Mux 
Change it accordingly.
b) Are you doing this for a specific platform or are they generic 3630 pin mux changes?
[Allen] No, not as of now.

- Allen

> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap- 
> owner@vger.kernel.org] On Behalf Of Menon, Nishanth
> Sent: Thursday, October 15, 2009 7:24 AM
> To: Pais, Allen; Aguirre Rodriguez, Sergio Alberto; linux-omap
> Cc: Chikkature Rajashekar, Madhusudhan; Pandita, Vikram; Gadiyar, 
> Anand; Cousson, Benoit; Felipe Balbi; Kevin Hilman; Premi, Sanjeev; 
> Shilimkar, Santosh; Tony Lindgren
> Subject: RE: [RFC][Patch V1] OMAP3: Mux Changes.
> 
> Please send your patch using git send email. And generate your patch 
> using git format patch
> 
> > -----Original Message-----
> > From: Pais, Allen
> > Sent: Thursday, October 15, 2009 4:38 AM
> >
> > Please ignore my previous mail.
> >
> > Muxes for OMAP 3630.
> >
> > Signed-off-by: Allen Pais <allen.pais@ti.com> diff --git
> a/arch/arm/mach-
> > omap2/mux.c b/arch/arm/mach-omap2/mux.c index b5fac32..93abb74 
> > 100644
> 
> 
> > --- a/arch/arm/mach-omap2/mux.c
> > +++ b/arch/arm/mach-omap2/mux.c
> > @@ -551,6 +551,42 @@ MUX_CFG_34XX("AF13_3430_MMC3_DAT3", 0x5e2, 
> > MUX_CFG_34XX("AF26_34XX_SYS_NIRQ", 0x1E0,
> >  		OMAP3_WAKEUP_EN | OMAP34XX_PIN_INPUT_PULLUP |
> >  		OMAP34XX_MUX_MODE0)
> > +
> > +/*Muxes for 3630 */
> > +MUX_CFG_34XX("H26_3630_DSS_DATA18", 0x100,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("H25_3630_DSS_DATA19", 0x102,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("E28_3630_DSS_DATA20", 0x104,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("J26_3630_DSS_DATA21", 0x106,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AC27_3630_DSS_DATA22", 0x108,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AC28_3630_DSS_DATA23", 0x10A,
> > +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> Why cant I do this in Board file?
I Really typed that??? Dumb me.. apologies on the noise.. I had meant bootloader..
> > +
> > +MUX_CFG_34XX("AF9_3630_ETKD8", 0x5EC,
> > +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AG9_3630_ETKD9", 0x5EE,
> > +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AG7_3630_ETKD12", 0x5F0,
> > +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> Why am I doing this?
> > +
> > +MUX_CFG_34XX("AA25_3630_UART2_TX", 0x178,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AD25_3630_UART2_RX", 0x17A,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AB25_3630_UART2_RTS", 0x176,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("AB26_3630_UART2_CTS", 0x174,
> > +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> Please explain..
> > +
> > +MUX_CFG_34XX("H20_UART3_RX_IRRX", 0x19E,
> > +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> > +MUX_CFG_34XX("H21_UART3_TX_IRTX", 0x1A0,
> > +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
> > +
> >  };
> ??
> >
> >  #define OMAP34XX_PINS_SZ	ARRAY_SIZE(omap34xx_pins)
> > diff --git a/arch/arm/plat-omap/include/mach/mux.h b/arch/arm/plat- 
> > omap/include/mach/mux.h index 0f49d2d..8d8cbe1 100644
> > --- a/arch/arm/plat-omap/include/mach/mux.h
> > +++ b/arch/arm/plat-omap/include/mach/mux.h
> > @@ -890,6 +890,32 @@ enum omap34xx_index {
> >
> >  	/* SYS_NIRQ T2 INT1 */
> >  	AF26_34XX_SYS_NIRQ,
> > +
> > +	/*Muxes for 3630*/
> > +	K28_3630_CAM_D6,
> > +	L28_3630_CAM_D7,
> > +	K27_3630_CAM_D8,
> > +	L27_3630_CAM_D9,
> > +
> > +	H26_3630_DSS_DATA18,
> > +	H25_3630_DSS_DATA19,
> > +	E28_3630_DSS_DATA20,
> > +	J26_3630_DSS_DATA21,
> > +	AC27_3630_DSS_DATA22,
> > +	AC28_3630_DSS_DATA23,
> > +
> > +	AF9_3630_ETKD8,
> > +	AG9_3630_ETKD9,
> > +	AG7_3630_ETK12,
> > +
> > +	AA25_3630_UART2_TX,
> > +	AD25_3630_UART2_RX,
> > +	AB25_3630_UART2_RTS,
> > +	AB26_3630_UART2_CTS,
> > +
> > +	H20_UART3_RX_IRRX,
> > +	H21_UART3_TX_IRTX,
> > +
> >  };
> >
> >  struct omap_mux_cfg {
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" 
> in the body of a message to majordomo@vger.kernel.org More majordomo 
> info at  http://vger.kernel.org/majordomo-info.html


Regards,
Nishanth Menon

Re: [RFC][Patch V1] OMAP3: Mux Changes.

[Nishanth Menon]

Pais, Allen had written, on 10/15/2009 11:53 PM, the following:
> a) A simple comment to all my comments: why cant we have these in bootloader and just simply leave the mux file alone?
> [Allen] Yes Nishanth, this would be a much cleaner approach. Even Santosh had suggested
> The same, if we can conclude on a approach here, I can go ahead and do the Mux 
> Change it accordingly.
Then lets please fix the bootloader and drop this patch.
> 
> - Allen
> 
>> -----Original Message-----
>> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap- 
>> owner@vger.kernel.org] On Behalf Of Menon, Nishanth
>> Sent: Thursday, October 15, 2009 7:24 AM
>> To: Pais, Allen; Aguirre Rodriguez, Sergio Alberto; linux-omap
Please try not to Top post. see [1]

-- 
Regards,
Nishanth Menon
Ref:
[1] http://www.arm.linux.org.uk/mailinglists/etiquette.php#e3

Re: [RFC][Patch V1] OMAP3: Mux Changes.

[Cory Maccarrone]

On Thu, Oct 15, 2009 at 9:53 PM, Nishanth Menon <nm@ti.com> wrote:
>
> Pais, Allen had written, on 10/15/2009 11:53 PM, the following:
>>
>> a) A simple comment to all my comments: why cant we have these in bootloader and just simply leave the mux file alone?
>> [Allen] Yes Nishanth, this would be a much cleaner approach. Even Santosh had suggested
>> The same, if we can conclude on a approach here, I can go ahead and do the Mux Change it accordingly.
>
> Then lets please fix the bootloader and drop this patch.
>>
>> - Allen
>>
>>> -----Original Message-----
>>> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap- owner@vger.kernel.org] On Behalf Of Menon, Nishanth
>>> Sent: Thursday, October 15, 2009 7:24 AM
>>> To: Pais, Allen; Aguirre Rodriguez, Sergio Alberto; linux-omap
>
> Please try not to Top post. see [1]
>
> --
> Regards,
> Nishanth Menon
> Ref:
> [1] http://www.arm.linux.org.uk/mailinglists/etiquette.php#e3

Maybe I'm missing something, but why is it more desirable to add the
mux code to the bootloader instead of the kernel?  Wouldn't adding it
to the kernel guarantee it works regardless of the bootloader?

- Cory

(Repost for linux-omap -- gmail had html enabled the first itme)

Re: [RFC][Patch V1] OMAP3: Mux Changes.

[Nishanth Menon]

Cory Maccarrone had written, on 10/16/2009 09:15 AM, the following:
> On Thu, Oct 15, 2009 at 9:53 PM, Nishanth Menon <nm@ti.com 
> <mailto:nm@ti.com>> wrote:
> 
>     Pais, Allen had written, on 10/15/2009 11:53 PM, the following:
> 
>         a) A simple comment to all my comments: why cant we have these
>         in bootloader and just simply leave the mux file alone?
>         [Allen] Yes Nishanth, this would be a much cleaner approach.
>         Even Santosh had suggested
>         The same, if we can conclude on a approach here, I can go ahead
>         and do the Mux Change it accordingly.
> 
>     Then lets please fix the bootloader and drop this patch.
> 
> 
[...]
> Maybe I'm missing something, but why is it more desirable to add the mux 
> code to the bootloader instead of the kernel?  Wouldn't adding it to the 
> kernel guarantee it works regardless of the bootloader?
adding mux in kernel guarentees that it is independent of the bootloader 
- yes - no questions on that. There are two strategies that can be taken:

A) Ensure that kernel is independent from bootloader variances
   - this is desirable and would adhere to the rules of an entity uses 
the resources(muxes) ONLY what it is required for itself to work -> e.g. 
bootloader would use the minimal set that is required for itself to 
function.

B) Divide the load -> always assume that kernel and bootloaders work 
together.
    - Do all the static muxes in u-boot -> since the u-boot dies off 
once the kernel starts up, the memory is completely freed up.
    - Only the dynamic muxes (e.g. pins being used in two modes by two 
drivers) are handled by the kernel.

The current strategy that is implemented for OMAP3 is (B), but once the 
kernel infrastructure for mux handling improves we should be able to 
arrive at a compromise between A and B as far as execution latencies Vs 
memory usage Vs independence issues are concerned. we are not there yet 
unfortunately.


-- 
Regards,
Nishanth Menon

Re: [RFC][Patch V1] OMAP3: Mux Changes.

[Tony Lindgren]

* Nishanth Menon <nm@ti.com> [091016 08:04]:
> Cory Maccarrone had written, on 10/16/2009 09:15 AM, the following:
>> On Thu, Oct 15, 2009 at 9:53 PM, Nishanth Menon <nm@ti.com  
>> <mailto:nm@ti.com>> wrote:
>>
>>     Pais, Allen had written, on 10/15/2009 11:53 PM, the following:
>>
>>         a) A simple comment to all my comments: why cant we have these
>>         in bootloader and just simply leave the mux file alone?
>>         [Allen] Yes Nishanth, this would be a much cleaner approach.
>>         Even Santosh had suggested
>>         The same, if we can conclude on a approach here, I can go ahead
>>         and do the Mux Change it accordingly.
>>
>>     Then lets please fix the bootloader and drop this patch.
>>
>>
> [...]
>> Maybe I'm missing something, but why is it more desirable to add the 
>> mux code to the bootloader instead of the kernel?  Wouldn't adding it 
>> to the kernel guarantee it works regardless of the bootloader?
> adding mux in kernel guarentees that it is independent of the bootloader  
> - yes - no questions on that. There are two strategies that can be taken:

Muxing in the bootloader only leads to impossible to debug bugs of type
"my device does not work" where the kernel has no clue what's going on.

In addition, at least the GPIO pins need to be dynamically remuxed
during the off-while-idle.

Also, if CONFIG_MUX is not set, the code is optimized out.

See the earlier discussion on the muxing. Basically we are going to move
to init time muxing in board-*.c files hopefully this coming merge window.
I'll post some patches probably next week on this.

Meanwhile, the current mux system works just fine, but is not suitable
for what we need for the dynamic GPIO pin muxing.

>
> A) Ensure that kernel is independent from bootloader variances
>   - this is desirable and would adhere to the rules of an entity uses  
> the resources(muxes) ONLY what it is required for itself to work -> e.g.  
> bootloader would use the minimal set that is required for itself to  
> function.
>
> B) Divide the load -> always assume that kernel and bootloaders work  
> together.
>    - Do all the static muxes in u-boot -> since the u-boot dies off once 
> the kernel starts up, the memory is completely freed up.
>    - Only the dynamic muxes (e.g. pins being used in two modes by two  
> drivers) are handled by the kernel.
>
> The current strategy that is implemented for OMAP3 is (B), but once the  
> kernel infrastructure for mux handling improves we should be able to  
> arrive at a compromise between A and B as far as execution latencies Vs  
> memory usage Vs independence issues are concerned. we are not there yet  
> unfortunately.

Once we have all the mux stuff reworked, we should move to model where
we mux all the pins during boot. This is with CONFIG_MUX set, without
that it will be all optimized out.

Regards,

Tony

Re: [RFC][Patch V1] OMAP3: Mux Changes.

[Tony Lindgren]

* Pais, Allen <allen.pais@ti.com> [091015 02:33]:
> Please ignore my previous mail.
> 
> Muxes for OMAP 3630.

Let's put this on hold and do it with the upcoming mux framework
instead. Adding all the missing 3630 mux settings should be easy
then.

Regards,

Tony

> 
> Signed-off-by: Allen Pais <allen.pais@ti.com> diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c index b5fac32..93abb74 100644
> --- a/arch/arm/mach-omap2/mux.c
> +++ b/arch/arm/mach-omap2/mux.c
> @@ -551,6 +551,42 @@ MUX_CFG_34XX("AF13_3430_MMC3_DAT3", 0x5e2,  MUX_CFG_34XX("AF26_34XX_SYS_NIRQ", 0x1E0,
>  		OMAP3_WAKEUP_EN | OMAP34XX_PIN_INPUT_PULLUP |
>  		OMAP34XX_MUX_MODE0)
> +
> +/*Muxes for 3630 */
> +MUX_CFG_34XX("H26_3630_DSS_DATA18", 0x100,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("H25_3630_DSS_DATA19", 0x102,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("E28_3630_DSS_DATA20", 0x104,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("J26_3630_DSS_DATA21", 0x106,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AC27_3630_DSS_DATA22", 0x108,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AC28_3630_DSS_DATA23", 0x10A,
> +		OMAP34XX_MUX_MODE3 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +
> +MUX_CFG_34XX("AF9_3630_ETKD8", 0x5EC,
> +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AG9_3630_ETKD9", 0x5EE,
> +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AG7_3630_ETKD12", 0x5F0,
> +		OMAP34XX_MUX_MODE1 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +
> +MUX_CFG_34XX("AA25_3630_UART2_TX", 0x178,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AD25_3630_UART2_RX", 0x17A,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AB25_3630_UART2_RTS", 0x176,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("AB26_3630_UART2_CTS", 0x174,
> +		OMAP34XX_MUX_MODE5 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +
> +MUX_CFG_34XX("H20_UART3_RX_IRRX", 0x19E,
> +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN) 
> +MUX_CFG_34XX("H21_UART3_TX_IRTX", 0x1A0,
> +		OMAP34XX_MUX_MODE2 | OMAP34XX_PIN_INPUT_PULLDOWN)
> +
>  };
>  
>  #define OMAP34XX_PINS_SZ	ARRAY_SIZE(omap34xx_pins)
> diff --git a/arch/arm/plat-omap/include/mach/mux.h b/arch/arm/plat-omap/include/mach/mux.h
> index 0f49d2d..8d8cbe1 100644
> --- a/arch/arm/plat-omap/include/mach/mux.h
> +++ b/arch/arm/plat-omap/include/mach/mux.h
> @@ -890,6 +890,32 @@ enum omap34xx_index {
>  
>  	/* SYS_NIRQ T2 INT1 */
>  	AF26_34XX_SYS_NIRQ,
> +
> +	/*Muxes for 3630*/
> +	K28_3630_CAM_D6,
> +	L28_3630_CAM_D7,
> +	K27_3630_CAM_D8,
> +	L27_3630_CAM_D9,
> +
> +	H26_3630_DSS_DATA18,
> +	H25_3630_DSS_DATA19,
> +	E28_3630_DSS_DATA20,
> +	J26_3630_DSS_DATA21,
> +	AC27_3630_DSS_DATA22,
> +	AC28_3630_DSS_DATA23,
> +
> +	AF9_3630_ETKD8,
> +	AG9_3630_ETKD9,
> +	AG7_3630_ETK12,
> +
> +	AA25_3630_UART2_TX,
> +	AD25_3630_UART2_RX,
> +	AB25_3630_UART2_RTS,
> +	AB26_3630_UART2_CTS,
> +
> +	H20_UART3_RX_IRRX,
> +	H21_UART3_TX_IRTX,
> +
>  };
>  
>  struct omap_mux_cfg {