Re: [PATCH V4 0/2] Add support for pci/non-pci probing for UFS

[Girish K S <girish.shivananjappa@xxxxxxxxxx>]

On 5 June 2012 18:32, Girish K S <girish.shivananjappa@xxxxxxxxxx> wrote:
> On 5 June 2012 15:11, S, Venkatraman <svenkatr@xxxxxx> wrote:
>> On Mon, Jun 4, 2012 at 5:15 PM, Girish K S
>> <girish.shivananjappa@xxxxxxxxxx> wrote:
>>> Hi maintainers,
>>> pls add
>>> Reviewed-by: Venkatraman S <svenkatr@xxxxxx>
>>> when you accept this patch
>>>
>> Girish,
>> Now that you have resent the patches, you could have added the
>> "Reviewed-by" tag yourself.
>> Anyway, there seems to be some problems with your mailer and the
>> patches have a lot of trailing ^M characters.
>> I also get this checkpatch warning:
>> WARNING: please write a paragraph that describes the config symbol fully
>> #29: FILE: drivers/scsi/ufs/Kconfig:45:
>>  config SCSI_UFSHCD
> i checked the patch again. This line is not modified by my patch.
> unable to understand the reason for it.
The checkpatch mandates the Kconfig's config option's description to
be min 4 lines.
I added 2 more lines of description to remove the warning.
will resend the patches
>>
>> James,
>>  These patched depend on some cleanups that I posted to the same
>> files a few weeks ago and now they are in Andrew Morton's queue. I'm
>> not very familiar with the process but I assumed it will get into
>> 3.5-rc1 as part of Andrew's pull request or equivalent.  Should I
>> resend them to you and ask it to be dropped from Andrew Morton's tree
>> ?
>>
>> Thanks,
>> Venkat.
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html