Re: [PATCH v3 2/4] mmc: support sdhci-pxav2.c

2011-06-07 Thread zhangfei gao
On Sat, Jun 4, 2011 at 8:04 AM, Philip Rakity prak...@marvell.com wrote: On Jun 3, 2011, at 11:42 AM, Philip Rakity wrote: On Jun 3, 2011, at 2:50 AM, Zhangfei Gao wrote:     SDHCI driver for pxav2 SoCs, such as pxa910, the driver based on sdhci-pltfm to handle resource etc.

Re: [PATCH v2 0/2] sdio: add quirk for spurious SDIO IRQ

2011-06-07 Thread Per Forlin
On 1 June 2011 17:59, Nicolas Pitre nicolas.pi...@linaro.org wrote: On Wed, 1 Jun 2011, Per Forlin wrote: From: Per Forlin per.for...@linaro.org Daniel Drake reported an issue in the libertas sdio client that was triggered by the sdio_single_irq functionality. His SDIO device seems to raise

Re: [PATCH v2 1/2] sdio: add quirk to handle pending IRQ in case of none CCCR_INTx bits

2011-06-07 Thread Per Forlin
On 1 June 2011 23:36, Troy Kisky troy.ki...@boundarydevices.com wrote: On 6/1/2011 1:48 AM, Per Forlin wrote: From: Per Forlin per.for...@linaro.org diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h index 7b4fd7b..852021c 100644 --- a/include/linux/mmc/card.h +++

Re: [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910

2011-06-07 Thread Arnd Bergmann
On Tuesday 07 June 2011 07:44:42 zhangfei gao wrote: On Sat, Jun 4, 2011 at 5:08 AM, Philip Rakity prak...@marvell.com wrote: Please update mmp2 and pxa defconfigs. The trend is replacing defconfig with device tree, it may not acceptable to update defconfig. Actually, the defconfigs are

Re: [PATCH 2/2] sdio: report error if pending IRQ but none function bits

2011-06-07 Thread Per Forlin
On 1 June 2011 16:30, Nicolas Pitre nicolas.pi...@linaro.org wrote: On Tue, 31 May 2011, Per Forlin wrote: Return error in case of pending IRQ but none functions bits in CCCR_INTx is set. Signed-off-by: Per Forlin per.for...@linaro.org ---  drivers/mmc/core/sdio_irq.c |    5 +  1

Re: [PATCH v3 2/4] mmc: support sdhci-pxav2.c

2011-06-07 Thread Arnd Bergmann
On Tuesday 07 June 2011 08:08:27 zhangfei gao wrote: +config MMC_SDHCI_PXAV2 + tristate Marvell PXAV2 SD Host Controller support + select MMC_SDHCI + select MMC_SDHCI_PLTFM + help + This selects the Marvell(R) PXAV2 SD Host Controller. + If you have a

Re: MMC runtime PM patches break libertas probe

2011-06-07 Thread Arnd Hannemann
Hi Ohad, Am 04.06.2011 00:52, schrieb Ohad Ben-Cohen: (cc'ing Arnd Hannermann) On Sat, Jun 4, 2011 at 1:28 AM, Bing Zhao bz...@marvell.com wrote: CMD5 Arg=0 refers to the very first CMD5 sent from host during initialization sequence. This is required because our state machine always

Re: [PATCH v3 2/4] mmc: support sdhci-pxav2.c

2011-06-07 Thread Philip Rakity
On Jun 7, 2011, at 12:31 AM, Eric Miao wrote: How about changing the title of the option though? A user will usually not know if he is using the v2 or v3 controller without reading the help text. If you change it to config MMC_SDHCI_PXAV2 tristate Marvell PXA168 SD Host Controller

Re: sdhci-pci fails on 3.0.0-rc1 on Dell E6510

2011-06-07 Thread Oliver Hartkopp
On 06.06.2011 21:56, Chris Ball wrote: Hi Oliver, On Mon, Jun 06 2011, Oliver Hartkopp wrote: dmesg is a bit more detailed: [6.242510] sdhci-pci :03:00.1: SDHCI controller found [1180:e822] (rev 3) [6.244168] sdhci-pci :03:00.1: PCI INT B - GSI 19 (level, low) - IRQ 19

Re: [PATCH] mmc: sdio: reset card during power_restore

2011-06-07 Thread Ohad Ben-Cohen
On Tue, Jun 7, 2011 at 7:41 PM, Daniel Drake d...@laptop.org wrote: Also, the most frustrating thing when working on this issue is that the codepaths for powering up a SD card between the following 3 situations are significantly different:  1. Power up and probe during boot  2. Resume from

Re: [PATCH] mmc: sdio: reset card during power_restore

2011-06-07 Thread Ohad Ben-Cohen
On Tue, Jun 7, 2011 at 7:41 PM, Daniel Drake d...@laptop.org wrote: On 5 June 2011 14:48, Ohad Ben-Cohen o...@wizery.com wrote: Sending a reset is necessary only if we are re-initing a powered on card, but in this case, we know that we just powered the card up, so this is not needed. Don't

Re: [PATCH v2 1/3] mmc: support sdhci-mmp2

2011-06-07 Thread zhangfei gao
On Wed, Jun 8, 2011 at 7:51 AM, Philip Rakity prak...@marvell.com wrote: On May 29, 2011, at 10:42 PM, zhangfei gao wrote: There is a little room for simplification, I think: +static unsigned int mmp2_get_ro(struct sdhci_host *host) +{ +     /* Micro SD does not support write-protect