Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1265141 > unrolled thread

Re: [PATCH v2 01/12] staging: rtl8188eu: if/else replaced by min_t

Started byGreg Kroah-Hartman <gregkh@linuxfoundation.org>
First post2015-11-08 18:20 +0100
Last post2015-11-10 10:20 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2 01/12] staging: rtl8188eu: if/else replaced by min_t Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-11-08 18:20 +0100
    Re: [PATCH v2 01/12] staging: rtl8188eu: if/else replaced by min_t Ivan Safonov <insafonov@gmail.com> - 2015-11-10 10:20 +0100

#1265141 — Re: [PATCH v2 01/12] staging: rtl8188eu: if/else replaced by min_t

FromGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Date2015-11-08 18:20 +0100
SubjectRe: [PATCH v2 01/12] staging: rtl8188eu: if/else replaced by min_t
Message-ID<qsBNg-30c-3@gated-at.bofh.it>
On Sun, Nov 08, 2015 at 02:07:11PM +0700, Ivan Safonov wrote:
> Duplicated code removed.
> 
> Signed-off-by: Ivan Safonov <insafonov@gmail.com>
> ---
>  drivers/staging/rtl8188eu/core/rtw_cmd.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)

You sent 3 versions of this patch, is this series the "real" one?  When
you resend, you need to say why you resent things, below the --- line
put the difference between v1 and v2 there so that we know what changed.

So I'm going to assume that the v2 series is the one you want to have
applied, if this isn't correct, please let me know.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1266345

FromIvan Safonov <insafonov@gmail.com>
Date2015-11-10 10:20 +0100
Message-ID<qtdfQ-39x-21@gated-at.bofh.it>
In reply to#1265141
On 11/09/2015 12:10 AM, Greg Kroah-Hartman wrote:
> On Sun, Nov 08, 2015 at 02:07:11PM +0700, Ivan Safonov wrote:
>> Duplicated code removed.
>>
>> Signed-off-by: Ivan Safonov <insafonov@gmail.com>
>> ---
>>   drivers/staging/rtl8188eu/core/rtw_cmd.c | 6 ++----
>>   1 file changed, 2 insertions(+), 4 deletions(-)
> You sent 3 versions of this patch, is this series the "real" one?  When
> you resend, you need to say why you resent things, below the --- line
> put the difference between v1 and v2 there so that we know what changed.
>
> So I'm going to assume that the v2 series is the one you want to have
> applied, if this isn't correct, please let me know.
>
> thanks,
>
> greg k-h

Yes, that's right, the second version is the only correct.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web