Hi Balaji,
I'm trying to reuse mmc_of_parse for parsing standard mmc features:
- cd-gpios / wp-gpios
- bus-width
- max-frequency
- keep-power-in-suspend
- enable-sdio-wakeup
- ti,non-removable, evtl.
Currently these are open-coded in of_get_hsmmc_pdata. I tried
removing them all from of_get_hsmmc_pdata and call mmc_of_parse
at the end of the probe function:
@@ -2241,8 +2264,6 @@ static int omap_hsmmc_probe(struct platform_device *pdev)
omap_hsmmc_protect_card(host);
+ mmc_of_parse(host->mmc);
mmc_add_host(mmc);
This should work transparently for most features except,
cd-gpios/wp-gpios. Pls can you have a look at the followup
patches? I tried an intermediate step mapping card detect /
read-only detect onto mmc/slot-gpio that are used by
mmc_of_parse. I don't have card detect/read-only detect pins, :-(
Do we still need cover_detect functionality or could it be merged
with card detect?
Another issue is 'ti,non-removable' which could be mappend to
'non-removable' if 'no_regulator_off_init' could be handled in a
different way.
if (of_find_property(np, "ti,non-removable", NULL)) {
pdata->slots[0].nonremovable = true;
pdata->slots[0].no_regulator_off_init = true;
}
thanks,
Andreas
Andreas Fenkart (3):
omap_hsmmc: reuse mmc/slot-gpio for write protect detection
omap_hsmmc: separate card_detect/cover detect logic
omap_hsmmc: reuse mmc/slot-gpio for card detect instead of open-coded
version
drivers/mmc/host/omap_hsmmc.c | 172 ++++++++++++++-------------------
include/linux/platform_data/mmc-omap.h | 6 +-
2 files changed, 73 insertions(+), 105 deletions(-)
--
2.0.0
--
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
