Re: [RFC PATCH] mmc: support background operation

2011-08-17 Thread J Freyensee
On 08/17/2011 07:11 PM, Jaehoon Chung wrote: Hi Jay.. I didn't know the tool 'sparse'. i will check the tool for your advice. Thank you a lot for your help.. :) here, I'll provide it from a write-up I did within my department. Unfortunately, finding documentation how to use it is pretty hard

Re: [RFC PATCH] mmc: support background operation

2011-08-17 Thread Jaehoon Chung
Hi Jay.. I didn't know the tool 'sparse'. i will check the tool for your advice. Thank you a lot for your help.. :) > Unfortunately, I am still a little new to this area of the Linux kernel > so I don't quite have the background to give an intelligent answer > w/respect to eMMC 4.41 and BKOPS. I

Re: [RFC PATCH] mmc: support background operation

2011-08-17 Thread J Freyensee
On 08/16/2011 09:03 PM, Jaehoon Chung wrote: Hi, J Freyensee wrote: On 08/12/2011 04:14 AM, Jaehoon Chung wrote: Hi mailing. This RFC patch is supported background operation(BKOPS). And if you want to test this patch, must apply "[PATCH v3] mmc: support HPI send command" This patch is based

SDIO

2011-08-17 Thread Veda N
To configure the wlan chip into 4 bit mode. What are sequence of commands that needs to be done by a host. Can one explain. -- Thanks & Regards, S. N. Veda -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo

about MicroSD transport issue for the current mainline kernel

2011-08-17 Thread seedshope
On 08/17/2011 09:51 PM, seedshope wrote: > Hi All, > > I uses potevio MicroSD 512M to test the SD/MMC driver in current master > kernel. I found some issue: > > 1. The issue > (1) After I insert the microSD,Why The SD card probe finish(The > parrition and device has already register). But The mor

Re: Testing Infrastructure for SD/MMC Commands

2011-08-17 Thread seedshope
Hi All, I uses potevio MicroSD 512M to test the SD/MMC driver in current master kernel. I found some issue: 1. The issue (1) After I insert the microSD,Why The SD card probe finish(The parrition and device has already register). But The more and more data transport between card and host? Who can

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Simon Horman
On Wed, Aug 17, 2011 at 02:22:56PM +0200, Guennadi Liakhovetski wrote: > On Wed, 17 Aug 2011, Simon Horman wrote: > > > On Wed, Aug 17, 2011 at 08:41:47PM +0900, Simon Horman wrote: > > > On Wed, Aug 17, 2011 at 01:24:58PM +0200, Guennadi Liakhovetski wrote: > > > > On Wed, 17 Aug 2011, Simon Horm

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Guennadi Liakhovetski
On Wed, 17 Aug 2011, Simon Horman wrote: > On Wed, Aug 17, 2011 at 08:41:47PM +0900, Simon Horman wrote: > > On Wed, Aug 17, 2011 at 01:24:58PM +0200, Guennadi Liakhovetski wrote: > > > On Wed, 17 Aug 2011, Simon Horman wrote: > > > > > > > Make use of per-source irq handles if the > > > > platfo

Re: [PATCH 1/2] mmc: add functions to enable/disable aggressive clock gating

2011-08-17 Thread Mika Westerberg
On Wed, Aug 17, 2011 at 09:51:31AM +0200, Linus Walleij wrote: > > I would suggest that in all patches using these functions, try > to replace: > > mmc_host_clk_disable() -> mmc_host_clk_ungate() > mmc_host_clk_enable() -> mmc_host_clk_gate() > Wow, that is indeed *much* cleaner way of doing th

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Simon Horman
On Wed, Aug 17, 2011 at 08:41:47PM +0900, Simon Horman wrote: > On Wed, Aug 17, 2011 at 01:24:58PM +0200, Guennadi Liakhovetski wrote: > > On Wed, 17 Aug 2011, Simon Horman wrote: > > > > > Make use of per-source irq handles if the > > > platform (data) has multiple irq sources. > > > > > > Also,

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Guennadi Liakhovetski
On Wed, 17 Aug 2011, Simon Horman wrote: > On Wed, Aug 17, 2011 at 01:24:58PM +0200, Guennadi Liakhovetski wrote: > > On Wed, 17 Aug 2011, Simon Horman wrote: > > > > > Make use of per-source irq handles if the > > > platform (data) has multiple irq sources. > > > > > > Also, as suggested by Gue

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Simon Horman
On Wed, Aug 17, 2011 at 01:24:58PM +0200, Guennadi Liakhovetski wrote: > On Wed, 17 Aug 2011, Simon Horman wrote: > > > Make use of per-source irq handles if the > > platform (data) has multiple irq sources. > > > > Also, as suggested by Guennadi Liakhovetski, > > add and use defines the index or

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Guennadi Liakhovetski
On Wed, 17 Aug 2011, Simon Horman wrote: > Make use of per-source irq handles if the > platform (data) has multiple irq sources. > > Also, as suggested by Guennadi Liakhovetski, > add and use defines the index or irqs in platform data. > > Cc: Guennadi Liakhovetski > Cc: Magnus Damm > Signed-o

[PATCH 1/4] mmc: tmio: Cache interrupt masks

2011-08-17 Thread Simon Horman
This avoids the need to look up the masks each time an interrupt is handled. As suggested by Guennadi. Cc: Guennadi Liakhovetski Cc: Magnus Damm Signed-off-by: Simon Horman --- * SDCARD portion tested on AP4/Mackerel * SDIO portion untested v3 * As suggested by Guennadi Liakhovetski - Onl

[PATCH 2/4] mmc: tmio: Provide separate interrupt handlers

2011-08-17 Thread Simon Horman
Provide separate interrupt handlers which may be used by platforms where SDHI has three interrupt sources. This patch also removes the commented-out handling of CRC and other errors. Cc: Guennadi Liakhovetski Cc: Magnus Damm Signed-off-by: Simon Horman --- * SDCARD portion tested on AP4/Mack

[PATCH 4/4] ARM: shmobile: ag5evm, ap4: Make use of irq index enum

2011-08-17 Thread Simon Horman
This is intended to make it easier to correctly order IRQs. As suggested by Guennadi Liakhovetski. Cc: Guennadi Liakhovetski Cc: Magnus Damm Cc: Paul Mundt Signed-off-by: Simon Horman --- Depends on "mmc: sdhi: Make use of per-source irq handlers" v4 * Update for corrected ordering of SH_M

[PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Simon Horman
Make use of per-source irq handles if the platform (data) has multiple irq sources. Also, as suggested by Guennadi Liakhovetski, add and use defines the index or irqs in platform data. Cc: Guennadi Liakhovetski Cc: Magnus Damm Signed-off-by: Simon Horman --- v5 * As suggested by Guennadi Lia

[PATCH 0/4 v5] mmc: tmio, sdhi: provide multiple irq handlers

2011-08-17 Thread Simon Horman
The SDHI driver already supports making use of up to three interrupt sources. This series breaks up the existing interrupt handler into three handlers, one for card access, one for card detect interrupts, and one for SDIO interrupts. A cover-all handler, which makes use of these new broken-out ha

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Simon Horman
On Wed, Aug 17, 2011 at 12:06:56PM +0200, Guennadi Liakhovetski wrote: > On Wed, 17 Aug 2011, Simon Horman wrote: > > > On Wed, Aug 17, 2011 at 10:20:24AM +0200, Guennadi Liakhovetski wrote: > > > On Wed, 17 Aug 2011, Simon Horman wrote: > > > > [snip ] > > > > > > + ret = irq = platform_g

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Guennadi Liakhovetski
On Wed, 17 Aug 2011, Simon Horman wrote: > On Wed, Aug 17, 2011 at 10:20:24AM +0200, Guennadi Liakhovetski wrote: > > On Wed, 17 Aug 2011, Simon Horman wrote: > > [snip ] > > > > + ret = irq = platform_get_irq(pdev, SH_MOBILE_SDHI_IRQ_SDCARD); > > > + if (irq >= 0) { > > > + multi_irq =

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Simon Horman
On Wed, Aug 17, 2011 at 10:20:24AM +0200, Guennadi Liakhovetski wrote: > On Wed, 17 Aug 2011, Simon Horman wrote: [snip ] > > + ret = irq = platform_get_irq(pdev, SH_MOBILE_SDHI_IRQ_SDCARD); > > + if (irq >= 0) { > > + multi_irq = true; > > + ret = request_irq(irq, tmio_mm

Re: [RFC PATCH] mmc: support background operation

2011-08-17 Thread Jaehoon Chung
Hi In eMMC spec, if set BKOPS_SUPPORT bit in ext_csd, BKOPS_EN, BKOPS_STATUS, BKOPS_START and URGENT_BKOPS are supported for default. So need not to send switch-command for BKOPS_EN when set BKOPS_SUPPORT bit. Thanks, Jaehoon Chung >> There is no code for enabling BKOPS_EN? >> If host want to h

Re: [PATCH 3/4] mmc: sdhi: Make use of per-source irq handlers

2011-08-17 Thread Guennadi Liakhovetski
On Wed, 17 Aug 2011, Simon Horman wrote: > Make use of per-source irq handles if the > platform (data) has multiple irq sources. > > Also, as suggested by Guennadi Liakhovetski, > add and use defines the index or irqs in platform data. > > Cc: Guennadi Liakhovetski > Cc: Magnus Damm > Signed-o

Re: [RFC PATCH] mmc: support background operation

2011-08-17 Thread Jaehoon Chung
Hi Seungwon. Seungwon Jeon wrote: > Hi Jaehoon Chung, > >> @@ -762,6 +773,23 @@ static int mmc_init_card(struct mmc_host *host, u32 >> ocr, >> } >> >> /* >> + * Enable HPI feature (if supported) >> + */ >> +if (card->ext_csd.hpi) { >> +err = mmc_switch(card, EXT_

RE: [RFC PATCH] mmc: support background operation

2011-08-17 Thread Seungwon Jeon
Hi Jaehoon Chung, Jaehoon Chung wrote: > > Hi mailing. > > This RFC patch is supported background operation(BKOPS). > And if you want to test this patch, must apply "[PATCH v3] mmc: support > HPI send command" > > This patch is based on Hanumath Prasad's patch "mmc: enable background > operatio

Re: [PATCH 2/2] mmc: prevent aggressive clock gating to race with ios updates

2011-08-17 Thread Linus Walleij
On Mon, Aug 15, 2011 at 12:03 PM, Mika Westerberg wrote: > We have seen at least two different races when clock gating kicks in in a > middle of ios structure update. > > First one happens when ios->clock is changed outside of aggressive clock > gating framework, for example via mmc_set_clock().

Re: [PATCH 1/2] mmc: add functions to enable/disable aggressive clock gating

2011-08-17 Thread Linus Walleij
On Mon, Aug 15, 2011 at 12:03 PM, Mika Westerberg wrote: > There are few places where we want to make sure that no clock gating takes > place. For example when we are updating several related fields in ios > structure and we don't want to accidentally pass the partially filled ios > to the host d