Re: [PATCH v2 1/3] dmaengine: add new dma API for max_segment_number

2011-06-07 Thread Dan Williams
On Tue, Jun 7, 2011 at 10:23 PM, FUJITA Tomonori wrote: > On Mon, 6 Jun 2011 11:48:56 -0700 > Dan Williams wrote: > >> dmaengine expands the class of dma providers to include standalone dma >> agents on a host bus (or elsewhere) in addition to the traditional bus >> mastering host-bus-adapters th

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

2011-06-07 Thread zhangfei gao
>> >> This method is used in v1, >> The method makes sdhci_ops non-const and have to dynamically alloc >> sdhci_ops, since host->ops is pointer. >> Will keep use const sdhci_ops, recommended by Arnd. > > > Not acceptable. > > > There are  3 cases for  WP signal > > a) They are connected to the cont

Re: [PATCH v2 1/3] dmaengine: add new dma API for max_segment_number

2011-06-07 Thread FUJITA Tomonori
On Mon, 6 Jun 2011 11:48:56 -0700 Dan Williams wrote: > dmaengine expands the class of dma providers to include standalone dma > agents on a host bus (or elsewhere) in addition to the traditional bus > mastering host-bus-adapters that the existing api understands. So in > the case of slave dma t

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

2011-06-07 Thread Philip Rakity
On Jun 7, 2011, at 7:56 PM, zhangfei gao wrote: > On Wed, Jun 8, 2011 at 7:51 AM, Philip Rakity 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

[PATCH V1] mmc: Enable the ADMA on esdhc imx driver

2011-06-07 Thread Richard Zhu
Eanble the ADMA2 mode on freescale esdhc imx driver, tested on MX51 and MX53. Only ADMA2 mode is enabled, MX25/35 can't support the ADMA2 mode. So this patch is only used for MX51/53. The ADMA2 mode supported or not can be distinguished by the Capability Register(offset 0x40) of eSDHC module. Up t

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

2011-06-07 Thread zhangfei gao
On Tue, Jun 7, 2011 at 11:04 PM, Philip Rakity wrote: > > 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 >>> >>> co

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 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

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

2011-06-07 Thread Philip Rakity
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 feature */ >>> + return 0; >>> +} >> >> You shouldn't nee

Re: [PATCH v2 1/3] dmaengine: add new dma API for max_segment_number

2011-06-07 Thread Dan Williams
On Mon, Jun 6, 2011 at 12:30 AM, Shawn Guo wrote: > Like dma_set(get)_max_seg_size for max_segment_size, the patch adds > max_segment_number into device_dma_parameters and creates the > corresponding dmaengine API dma_set(get)_max_seg_number for it. > > Here is the user story that tells the need o

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

2011-06-07 Thread Yinghai Lu
thanks. second report. can you send out whole boot log. Yinghai On 06/07/2011 01:06 PM, Oliver Hartkopp wrote: > On 07.06.2011 20:36, Oliver Hartkopp wrote: > Hi all, > > the commit "PCI: update bridge resources to get more big ranges when > allocating space (again)" > > http://git.kernel.or

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 wrote: > On 5 June 2011 14:48, Ohad Ben-Cohen 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 ask me for an explanati

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 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 suspend >  3.

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

[PATCH] libertas: add sd8686 reset_card support

2011-06-07 Thread Daniel Drake
At http://dev.laptop.org/ticket/10748 we are seeing a case of the libertas firmware randomly stopping responding to commands after resume. Careful monitoring of communications indicates a firmware or hardware bug, which has been reported to Marvell. Work around this issue by adding a reset_card me

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

2011-06-07 Thread Daniel Drake
On 5 June 2011 14:48, Ohad Ben-Cohen wrote: >> + >> +       /* >> +        * Reset the card by performing the same steps that are taken by >> +        * mmc_rescan_try_freq() and mmc_attach_sdio() during a "normal" >> probe >> +        */ >> +       sdio_reset(host); > > Sending a reset is necess

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

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 wrote: >> "CMD5 Arg=0" refers to the very first CMD5 sent from host during >> initialization sequence. >> This is required because our state machine always expects two

Re: MMC runtime PM patches break libertas probe

2011-06-07 Thread Ohad Ben-Cohen
On Tue, Jun 7, 2011 at 5:34 PM, Arnd Hannemann wrote: > Sorry, I don't have the hardware anymore so I'm not able to check this. Ok, thanks for the reply ! -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo i

sdhci.c after suspend, my sd host controller cannot work normally,

2011-06-07 Thread yingchun li
Hi, I have an android box, and the kernel is 2.6.32. and my sd card host driver use the sdhci driver. now I have a problem when sdhost suspend and resume. >From the log, after the system suspend, just before sdhost resume, there is a sdhci_request io happen. in the sdhci_request, it st

[PATCH V1] mmc: Enable the ADMA on esdhc imx driver

2011-06-07 Thread Richard Zhu
Eanble the ADMA2 mode on freescale esdhc imx driver, tested on MX51 and MX53. Only ADMA2 mode is enabled, MX25/35 can't support the ADMA2 mode. So this patch is only used for MX51/53. The ADMA2 mode supported or not can be distinguished by the Capability Register(offset 0x40) of eSDHC module. Up t

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

2011-06-07 Thread Eric Miao
> 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 support (PXAV2)" >        default CPU_PXA16

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. > >> +

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 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 >> --- >>  drivers/mmc/core/sdio_irq.c |    5 + >>  1 files changed, 5 insertions(+), 0

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 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 not going aw