From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaehoon Chung Subject: Re: [PATCH v9] mmc: support BKOPS feature for eMMC Date: Thu, 12 Jul 2012 14:02:16 +0900 Message-ID: <4FFE5A58.1060807@samsung.com> References: <4FD1821D.4070603@samsung.com> <4FD9EB47.6000905@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout4.samsung.com ([203.254.224.34]:17372 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751857Ab2GLFC1 (ORCPT ); Thu, 12 Jul 2012 01:02:27 -0400 Received: from epcpsbgm2.samsung.com (mailout4.samsung.com [203.254.224.34]) by mailout4.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0M71003SM7C2CUN0@mailout4.samsung.com> for linux-mmc@vger.kernel.org; Thu, 12 Jul 2012 14:02:26 +0900 (KST) Received: from [10.90.51.55] by mmp1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0M7100D217BYH770@mmp1.samsung.com> for linux-mmc@vger.kernel.org; Thu, 12 Jul 2012 14:02:25 +0900 (KST) In-reply-to: <4FD9EB47.6000905@intel.com> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Adrian Hunter Cc: Jaehoon Chung , linux-mmc , Chris Ball , Kyungmin Park , Hanumath Prasad , Per FORLIN , Sebastian Rasmussen , "Dong, Chuanxiao" , "svenkatr@ti.com" , Saugata Das , Konstantin Dorfman , Maya Erez , Ulf Hansson Hi, Adrian, On 06/14/2012 10:46 PM, Adrian Hunter wrote: > On 08/06/12 07:39, 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 BKOPS, first issue HPI to interrupt the ongoing BKOPS >> and then service the request. >> If BKOPS-STATUS is upper than LEVEL2, need to check until clear >> the BKOPS-STATUS vaule. >> >> If you want to enable this feature, set MMC_CAP2_BKOPS. >> And if you want to set the BKOPS_EN bit in ext_csd register, >> use the MMC_CAP2_INIT_BKOPS. >> >> Future considerations >> * Check BKOPS_LEVEL=1 and start BKOPS in a preventive manner. >> * Interrupt ongoing BKOPS before powering off the card. >> * How get BKOPS_STATUS value.(periodically send ext_csd command?) >> >> Signed-off-by: Jaehoon Chung >> Signed-off-by: Kyungmin Park >> Signed-off-by: Konstantin Dorfman >> Signed-off-by: Maya Erez >> --- > > I would not expect this to work nicely with runtime PM. I expect that BKOPS > would need to be stopped beforehand. But that would limit the time > available for BKOPS since runtime PM would always kick in and stop it. How > is runtime PM to be handled? I think that add some function like the mmc_runtime_pm_suspend(). int mmc_runtime_pm_suspend() { if running bkops { waiting for limit time..=> (when upper than Level2) otherwise don't wait stop-bkops } else nothing.. } Almost runtime PM is controlled at host side. So it's not very good that add the bkops control code at host side. How about this? i want to get your opinion. Best Regards, Jaehoon Chung > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >