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

2009-11-11 Thread Tony Lindgren
* Pais, Allen allen.p...@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.p...@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 {
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2009-10-16 Thread 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 n...@ti.com 
mailto:n...@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
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2009-10-16 Thread Tony Lindgren
* Nishanth Menon n...@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 n...@ti.com  
 mailto:n...@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
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2009-10-15 Thread Pais, Allen
Please ignore my previous mail.

Muxes for OMAP 3630.

Signed-off-by: Allen Pais allen.p...@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 {--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2009-10-15 Thread 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.p...@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 {
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2009-10-15 Thread 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-ow...@vger.kernel.org [mailto:linux-omap- 
 ow...@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.p...@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 majord...@vger.kernel.org More majordomo 
 info at  http://vger.kernel.org/majordomo-info.html


Regards,
Nishanth Menon

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2009-10-15 Thread 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-ow...@vger.kernel.org [mailto:linux-omap- 
ow...@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
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html