Re: [PATCH 0/2] mmc: rtsx: add new cmd type handle and modify errorhandle

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 26 March 2014 02:29, micky <micky_ching@xxxxxxxxxxxxxx> wrote:
> Hi Ulf,
>
> On 03/25/2014 06:44 PM, Ulf Hansson wrote:
>>
>> On 25 March 2014 10:47,  <micky_ching@xxxxxxxxxxxxxx> wrote:
>>>
>>> From: Micky Ching <micky_ching@xxxxxxxxxxxxxx>
>>>
>>> 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 <mmc: rtsx: fix bug and support nonblocking request>
> see: https://lkml.org/lkml/2014/2/17/270
> checkpatch.pl seems ok, is there any thing I need to check?

It seems like the checkpatch script has been recently patched. When I
ran the script for your patch I used the latest from Stephen
Rothwell's linux-next tree. Please do the same.

Kind regards
Ulf Hansson

>
> 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
>>
>> .
>>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux