RE: [PATCH 1/1] Default MUX configuration added - GPIO140-143, GPIO0 and GPIO9

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Vikram,

Pls find my comments inlined.



-----Original Message-----
From: Pandita, Vikram 
Sent: Tuesday, December 16, 2008 11:08 AM
To: Pillai, Manikandan; linux-omap@xxxxxxxxxxxxxxx
Subject: RE: [PATCH 1/1] Default MUX configuration added - GPIO140-143, GPIO0 and GPIO9


>-----Original Message-----
>From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of Pillai,
>Manikandan
>Sent: Monday, December 15, 2008 11:31 PM
>To: linux-omap@xxxxxxxxxxxxxxx
>Cc: Pillai, Manikandan
>Subject: [PATCH 1/1] Default MUX configuration added - GPIO140-143, GPIO0 and GPIO9
>
>Default MUX configurations for GPIO on OMAP3 EVM boards are added.
>Fixed for "_UP" naming convention for GPIOs comment.

Patch is fine but looks like 
[Pillai, Manikandan] OK
Mux framework seems to be not efficient.
[Pillai, Manikandan] It's not efficient but I have just added the requirments for the new power board in the existing framework. Revamping the whole MUX logic
would be a good amount of work.

"There is one GLOBAL table for all omap devices." 
How maintainable is that? 
Also care has to be taken to match the enum position with mux.c array location. This is error prone.

Mux tables should be per device basis.
Any thoughts.


>
>Signed-off-by: Manikandan Pillai <mani.pillai@xxxxxx>
>---
> arch/arm/mach-omap2/mux.c             |   13 +++++++++++++
> arch/arm/plat-omap/include/mach/mux.h |    6 ++++++
> 2 files changed, 19 insertions(+), 0 deletions(-)
>
>diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c
>index dacb41f..1556688 100644
>--- a/arch/arm/mach-omap2/mux.c
>+++ b/arch/arm/mach-omap2/mux.c
>@@ -459,6 +459,19 @@ MUX_CFG_34XX("AH8_34XX_GPIO29", 0x5fa,
> 		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_INPUT)
> MUX_CFG_34XX("J25_34XX_GPIO170", 0x1c6,
> 		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_INPUT)
>+MUX_CFG_34XX("AF26_34XX_GPIO0", 0x1e0,
>+		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_OUTPUT)
>+MUX_CFG_34XX("AF22_34XX_GPIO9", 0xa18,
>+		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_OUTPUT)
>+MUX_CFG_34XX("AF6_34XX_GPIO140_UP", 0x16c,
>+		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_INPUT_PULLUP)
>+MUX_CFG_34XX("AE6_34XX_GPIO141", 0x16e,
>+		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_OUTPUT)
>+MUX_CFG_34XX("AF5_34XX_GPIO142", 0x170,
>+		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_OUTPUT)
>+MUX_CFG_34XX("AE5_34XX_GPIO143", 0x172,
>+		OMAP34XX_MUX_MODE4 | OMAP34XX_PIN_OUTPUT)
>+
> };
>
> #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 f4362b8..67fddec 100644
>--- a/arch/arm/plat-omap/include/mach/mux.h
>+++ b/arch/arm/plat-omap/include/mach/mux.h
>@@ -790,6 +790,12 @@ enum omap34xx_index {
> 	 */
> 	AH8_34XX_GPIO29,
> 	J25_34XX_GPIO170,
>+	AF26_34XX_GPIO0,
>+	AF22_34XX_GPIO9,
>+	AF6_34XX_GPIO140_UP,
>+	AE6_34XX_GPIO141,
>+	AF5_34XX_GPIO142,
>+	AE5_34XX_GPIO143
> };
>
> struct omap_mux_cfg {
>--
>1.5.6
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>the body of a message to majordomo@xxxxxxxxxxxxxxx
>More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux