[PATCH v2] OMAP4: Pandaboard: Fix MMC card detect gpio line

Subsystems: arm port, omap2+ support, the rest

STALE5714d REVIEWED: 7 (6M)

2 review trailers.

2 messages, 2 authors, 2010-12-22 · open the first message on its own page

[PATCH v2] OMAP4: Pandaboard: Fix MMC card detect gpio line

From: nm@ti.com (Nishanth Menon)
Date: 2010-12-21 14:14:34

From: Raghuveer Murthy <redacted>

commit bf56f0a6668cd (2.6.37-rc1), from Nishanth Menon attempted
to fix card detection for PandaBoard, unfortunately, the fix missed
to initialize .gpio_cd member of omap2_hsmmc_info. This results
in a default value of '0', which is a valid GPIO line.
On PandaBoard, the side effect of this is that GPIO line 0 controls
the powering TFP410 DVI chip, and without the fix DVI chip is
inadvertently powered.

Tested-by: David Anders <redacted>

Acked-by: Nishanth Menon <nm@ti.com>

Signed-off-by: Kishore Kadiyala <redacted>
Signed-off-by: Raghuveer Murthy <redacted>
---
v2: updated comment description to point out the potential regression.
   it might be nice to have this in .37 rc cycle as the change was
   introduced in .37-rc1, however, the DVI functionality(which is impacted
   as a side effect) is not enabled in .37 rc.

v1: http://marc.info/?t=129172419600002&r=1&w=2
 arch/arm/mach-omap2/board-omap4panda.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap4panda.c b/arch/arm/mach-omap2/board-omap4panda.c
index 1ecd0a6..da24745 100644
--- a/arch/arm/mach-omap2/board-omap4panda.c
+++ b/arch/arm/mach-omap2/board-omap4panda.c
@@ -142,6 +142,7 @@ static struct omap2_hsmmc_info mmc[] = {
 		.mmc		= 1,
 		.caps		= MMC_CAP_4_BIT_DATA | MMC_CAP_8_BIT_DATA,
 		.gpio_wp	= -EINVAL,
+		.gpio_cd	= -EINVAL,
 	},
 	{}	/* Terminator */
 };
-- 
1.6.3.3

[PATCH v2] OMAP4: Pandaboard: Fix MMC card detect gpio line

From: tony@atomide.com (Tony Lindgren)
Date: 2010-12-22 01:57:27

* Nishanth Menon [off-list ref] [101221 06:14]:
From: Raghuveer Murthy <redacted>

commit bf56f0a6668cd (2.6.37-rc1), from Nishanth Menon attempted
to fix card detection for PandaBoard, unfortunately, the fix missed
to initialize .gpio_cd member of omap2_hsmmc_info. This results
in a default value of '0', which is a valid GPIO line.
On PandaBoard, the side effect of this is that GPIO line 0 controls
the powering TFP410 DVI chip, and without the fix DVI chip is
inadvertently powered.

Tested-by: David Anders <redacted>

Acked-by: Nishanth Menon <nm@ti.com>

Signed-off-by: Kishore Kadiyala <redacted>
Signed-off-by: Raghuveer Murthy <redacted>
---
v2: updated comment description to point out the potential regression.
   it might be nice to have this in .37 rc cycle as the change was
   introduced in .37-rc1, however, the DVI functionality(which is impacted
   as a side effect) is not enabled in .37 rc.
I'll queue this for .38 because of "it never worked earlier either" logic.

Thanks,

Tony
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help