From mboxrd@z Thu Jan 1 00:00:00 1970 From: micky Subject: Re: [PATCH 0/2] mmc: rtsx: add new cmd type handle and modify errorhandle Date: Wed, 26 Mar 2014 09:29:37 +0800 Message-ID: <53322D81.3050701@realsil.com.cn> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: driverdev-devel-bounces@linuxdriverproject.org To: Ulf Hansson Cc: Samuel Ortiz , Greg Kroah-Hartman , linux-mmc , "linux-kernel@vger.kernel.org" , Chris Ball , Wei WANG , Roger , devel@linuxdriverproject.org, Dan Carpenter List-Id: linux-mmc@vger.kernel.org Hi Ulf, On 03/25/2014 06:44 PM, Ulf Hansson wrote: > On 25 March 2014 10:47, wrote: >> From: Micky Ching >> >> Add new command type(R1 without CRC) handle, without this >> patch mmc card initialize will be failed. >> >> Using a more careful handle in request timeout, this would >> improve error recover capability. Debug info is printed >> using non DMA mode, this would help print more accurately >> for DMA command failed. Smatch warning was removed. > Hi Micky, > > Seems like this patchset has been around for some time, sorry for the delay. > > I don't have any objections to it getting merged, but you need to run > checkpatch of your patches. Patch2 has a warning you should fix. Patch2 is based on see: https://lkml.org/lkml/2014/2/17/270 checkpatch.pl seems ok, is there any thing I need to check? Best Regards. micky. > Kind regards > Ulf Hansson > > >> Micky Ching (2): >> mmc: rtsx: add R1-no-CRC mmc command type handle >> mmc: rtsx: modify error handle and remove smatch warnings >> >> drivers/mmc/host/rtsx_pci_sdmmc.c | 121 ++++++++++++++++++++----------------- >> 1 file changed, 67 insertions(+), 54 deletions(-) >> >> -- >> 1.7.9.5 > . >