Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-09 Thread Pierre Tardy
On Fri, Jan 7, 2011 at 12:16 AM, Ohad Ben-Cohen wrote: > Hi Pierre, > > On Wed, Jan 5, 2011 at 2:28 PM, Pierre Tardy wrote: >> Well... TI is selling quirky hw, would make sense that you handle this. :-) > > Heh I'd happily take over - just say the word :) Well anyway, I needed this for my power/c

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-06 Thread Ohad Ben-Cohen
Hi Pierre, On Wed, Jan 5, 2011 at 2:28 PM, Pierre Tardy wrote: > Well... TI is selling quirky hw, would make sense that you handle this. :-) Heh I'd happily take over - just say the word :) > Subject: [PATCH] mmc: add per device quirk placeholder > > Some cards have quirks valid for every platf

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-05 Thread Pierre Tardy
On Wed, Jan 5, 2011 at 10:40 AM, Ohad Ben-Cohen wrote: > On Wed, Jan 5, 2011 at 11:10 AM, Pierre Tardy wrote: >> Well, we dont do it like this in intel_mid. Every platform quirk has >> to come somehow from the firmware (SFI), and there is no such >> descriptor to my knowledge. Lets see what John

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-05 Thread Ohad Ben-Cohen
On Wed, Jan 5, 2011 at 11:10 AM, Pierre Tardy wrote: > Well, we dont do it like this in intel_mid. Every platform quirk has > to come somehow from the firmware (SFI), and there is no such > descriptor to my knowledge. Lets see what John is proposing... I think that John shared it with you this mo

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-05 Thread Pierre Tardy
On Wed, Jan 5, 2011 at 12:34 AM, Ohad Ben-Cohen wrote: > On Wed, Jan 5, 2011 at 1:25 AM, Pierre Tardy wrote: >> There is no such platform data in intel_mid platforms.. Sound like >> another platform hack will be needed for wl1271 to be integrated. :-( > > John (cc'ed) ported this to your platform

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-04 Thread Ohad Ben-Cohen
On Wed, Jan 5, 2011 at 1:25 AM, Pierre Tardy wrote: > There is no such platform data in intel_mid platforms.. Sound like > another platform hack will be needed for wl1271 to be integrated. :-( John (cc'ed) ported this to your platform and can share the solution with you; IIRC the idea was the sam

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-04 Thread Pierre Tardy
>  static struct omap2_hsmmc_info mmc[] __initdata = { >        { >                .name           = "external", > @@ -221,6 +233,7 @@ static struct omap2_hsmmc_info mmc[] __initdata = { >                .gpio_wp        = -EINVAL, >                .gpio_cd        = -EINVAL, >                .nonrem

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-04 Thread Ohad Ben-Cohen
Hi Pierre, On Wed, Jan 5, 2011 at 12:48 AM, Pierre Tardy wrote: > This sounds good, and may solve this problem we have with wl1271. > Then, how do you tell the mmc stack the wl1271 has the quirk > MMC_QUIRK_NONSTD_FUNC_IF? Something like this: diff --git a/arch/arm/mach-omap2/board-zoom-periphe

Re: [PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-04 Thread Pierre Tardy
On Tue, Jan 4, 2011 at 10:10 PM, Ohad Ben-Cohen wrote: > Use MMC_QUIRK_NONSTD_FUNC_IF to ignore the "SDIO Standard Function interface > code" as indicated by the card's FBR, and instead treat all functions as > non-standard interfaces. Hi Ohad, This sounds good, and may solve this problem we have

[PATCH 4/4] mmc: add MMC_QUIRK_NONSTD_FUNC_IF

2011-01-04 Thread Ohad Ben-Cohen
Use MMC_QUIRK_NONSTD_FUNC_IF to ignore the "SDIO Standard Function interface code" as indicated by the card's FBR, and instead treat all functions as non-standard interfaces. This is required to prevent standard drivers (e.g. btsdio) from facing errors when trying to communicate with SDIO cards th