Thread (16 messages) 16 messages, 4 authors, 2010-11-25
STALE5679d

[PATCH] [OMAP] HTCHERALD: MMC, I2C, HTCPLD, SPI, TSC2046

From: Michał Mirosław <hidden>
Date: 2010-09-30 19:05:51
Also in: linux-omap

2010/9/30 Tony Lindgren [off-list ref]:
* Cory Maccarrone [off-list ref] [100930 11:34]:
quoted
quoted
Looks like also board-sx1-mmc.c and board-h[23]-mmc.c have the
same spotty voltage range.
Cory, care to do a patch that fixes it for all of them?
Yeah, I can do that. ?I'll resubmit this patch too with the fixed up ranges.
Turns out I already did it :) Care to test/ack this one?
[...]
quoted hunk ↗ jump to hunk
diff --git a/arch/arm/mach-omap1/board-sx1-mmc.c b/arch/arm/mach-omap1/board-sx1-mmc.c
index 5b33ae8..be5a365 100644
--- a/arch/arm/mach-omap1/board-sx1-mmc.c
+++ b/arch/arm/mach-omap1/board-sx1-mmc.c
@@ -44,7 +44,8 @@ static struct omap_mmc_platform_data mmc1_data = {
? ? ? ?.nr_slots ? ? ? ? ? ? ? ? ? ? ? = 1,
? ? ? ?.slots[0] ? ? ? = {
? ? ? ? ? ? ? ?.set_power ? ? ? ? ? ? ?= mmc_set_power,
- ? ? ? ? ? ? ? .ocr_mask ? ? ? ? ? ? ? = MMC_VDD_28_29 | MMC_VDD_30_31 |
+ ? ? ? ? ? ? ? .ocr_mask ? ? ? ? ? ? ? = MMC_VDD_28_29 | MMC_VDD_29_30 |
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? MMC_VDD_30_31 | MMC_VDD_31_32 |
? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?MMC_VDD_32_33 | MMC_VDD_33_34,
? ? ? ? ? ? ? ?.name ? ? ? ? ? ? ? ? ? = "mmcblk",
? ? ? ?},
[...]

Al least this one seems wrong (haven't checked others) as the
mmc_set_power() ignores vdd parameter. This suggests that the board
supports only one particular voltage, not the whole range.

Best Regards,
Micha? Miros?aw
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help