Re: [PATCHv2 00/16] [FS, MM, block, MMC]: eMMC High Priority Interrupt Feature

2012-05-11 Thread S, Venkatraman
On Wed, May 9, 2012 at 6:15 AM, Minchan Kim wrote: > On 05/09/2012 01:31 AM, S, Venkatraman wrote: > >> On Tue, May 8, 2012 at 1:16 PM, Minchan Kim wrote: >>> On 05/03/2012 11:22 PM, Venkatraman S wrote: >>> Standard eMMC (Embedded MultiMedia Card) specification expects to execute one r

Re: [PATCH v8] mmc: support BKOPS feature for eMMC

2012-05-11 Thread Subhash Jadavani
Hi Jaehoon, Please find comments inline below. On 5/10/2012 7:07 PM, Jaehoon Chung wrote: Enable eMMC background operations (BKOPS) feature. If URGENT_BKOPS is set after a response, note that BKOPS are required. After all I/O requests are finished, run BKOPS if required. Should read/write oper

Re: [PATCH v3 1/2] MMC-4.5 Power OFF Notify rework

2012-05-11 Thread Subhash Jadavani
Hi Girish, On 5/7/2012 7:11 PM, Girish K S wrote: This is a rework of the existing POWER OFF NOTIFY patch. The current problem with the patch comes from the ambiguity on the usage of POWER OFF NOTIFY together with SLEEP and misunderstanding on the usage of MMC_POWER_OFF power_mode from mmc_set_i

Re: [PATCH v8] mmc: support BKOPS feature for eMMC

2012-05-11 Thread S, Venkatraman
On Thu, May 10, 2012 at 7:07 PM, Jaehoon Chung wrote: > Enable eMMC background operations (BKOPS) feature. > > If URGENT_BKOPS is set after a response, note that BKOPS > are required. After all I/O requests are finished, run > BKOPS if required. Should read/write operations be requested > during B

Re: [PATCH 4/4] mmc: core: Send HPI only till it is successful

2012-05-11 Thread S, Venkatraman
On Wed, May 9, 2012 at 8:20 PM, Philip Rakity wrote: > > On Apr 13, 2012, at 7:39 AM, Namjae Jeon wrote: > >> Hi. Venkatraman. >> >> You fixed 10 times. why is it 10 times ? >> and checking err from mmc_send_status is not needed ? is it  also >> infinite case ? >> >> Thanks. >> >> 2012/4/13 Venkat

Re: [PATCH v8] mmc: support BKOPS feature for eMMC

2012-05-11 Thread kdorfman
> Hi Konstantin. > > On 05/10/2012 11:46 PM, kdorf...@codeaurora.org wrote: > >> Hi Jaehoon, >> I see 2 issues with new patch: >> 1. New code starts BKOPS always with waiting for BUSY. (see comment >> below >> in the code) >> 2. Runtime suspend (mmc_suspend_host) do not checks that BKOPS is in >> p

Re: [PATCH 3/3] mmc: sdhci-imx-esdhc: convert to use pinctrl subsystem

2012-05-11 Thread Dong Aisheng
On Fri, May 11, 2012 at 03:36:56PM +0800, Shawn Guo wrote: > On Wed, May 02, 2012 at 07:31:22PM +0800, Dong Aisheng wrote: > > From: Dong Aisheng > > > > Acked-by: Stephen Warren > > Signed-off-by: Dong Aisheng > > --- > > ChangeLog v4->v5: > > *split arch/arm/mach-imx/* changes into a separat

Re: [PATCH 3/3] mmc: sdhci-imx-esdhc: convert to use pinctrl subsystem

2012-05-11 Thread Shawn Guo
On Wed, May 02, 2012 at 07:31:22PM +0800, Dong Aisheng wrote: > From: Dong Aisheng > > Acked-by: Stephen Warren > Signed-off-by: Dong Aisheng > --- > ChangeLog v4->v5: > *split arch/arm/mach-imx/* changes into a separate patch > ChangeLog v3->v4: > * call provide dummies in imx*_soc_init(). >