[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <554B7884.90002@kerneldev.net>
Date: Thu, 07 May 2015 09:36:52 -0500
From: Jaime Arrocha <jarrocha@...neldev.net>
To: Sudip Mukherjee <sudipm.mukherjee@...il.com>
CC: Jaime Arrocha <jarr@...neldev.net>, greg@...ah.com,
abbotti@....co.uk, hsweeten@...ionengravers.com,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: comedi: coding style identation error fix
On 05/07/2015 08:21 AM, Sudip Mukherjee wrote:
> On Thu, May 07, 2015 at 08:01:53AM -0500, Jaime Arrocha wrote:
>>
>>
>> On 05/07/2015 12:34 AM, Sudip Mukherjee wrote:
>>> On Wed, May 06, 2015 at 05:13:41PM -0500, Jaime Arrocha wrote:
>>>> Errors found by checkpatch.pl.
>>>> ERROR: code indent should use tabs where possible
>>>> /drivers/staging/comedi/drivers/das16m1.c:49
>>>> /drivers/staging/comedi/drivers/das16m1.c:50
>>>>
>>>> Signed-off-by: Jaime Arrocha <jarr@...neldev.net>
>>>> ---
>>> you are sending this patch second time with different subject.
>>> any reason for that?
>>>
>>> regards
>>> sudip
>>>
>>
>> Yes, the first one had the subject line too long and not clear.
>> When I need to resend a patch like I did, should I add on the second
>> one the reason behind resending and dismissing the first one? Thank
>> you.
>
> first of all you should reply to the first patch stating that should not
> be applied and that you have sent a second patch. And this one should
> have had a v2 in the subject with the reason of v2 below the --- cutoff.
> since you have already sent the second patch just reply to the fist
> patch as of now.
> And you have sent the patches to greg@...ah.com, it should have been
> gregkh@...uxfoundation.org . always use get_maintainer.pl to see whom
> to send patches.
>
> regards
> sudip
>
>
Thanks Sudip, I will dismiss this one too and do it correctly. I
accidentally took Greg's email from the TODO file.
This patch should not be applied.
Thanks,
jaime
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists