From: Greg KH <gregkh@linuxfoundation.org>
To: Harsha Sharma <harshasharmaiitr@gmail.com>
Cc: outreachy-kernel <outreachy-kernel@googlegroups.com>
Subject: Re: [Outreachy kernel] Re: [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space
Date: Mon, 11 Sep 2017 12:34:10 -0700 [thread overview]
Message-ID: <20170911193410.GD23729@kroah.com> (raw)
In-Reply-To: <4485e4d2-138f-4c7a-aefb-df6a8ce841e2@googlegroups.com>
On Mon, Sep 11, 2017 at 11:59:23AM -0700, Harsha Sharma wrote:
>
You are still using html email, and you stripped off the email of the
person who originally sent you the message, which wasn't good :(
> On Monday, September 11, 2017 at 10:16:58 PM UTC+5:30, gregkh wrote:
>
> On Sun, Sep 10, 2017 at 11:04:34PM +0530, Harsha Sharma wrote:
> > Remove space between function name and open parenthesis '('
> >
> > Signed-off-by: Harsha Sharma <harshash...@gmail.com>
> > ---
> > �drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++---
> > �1 file changed, 3 insertions(+), 3 deletions(-)
>
> This and patch 8/10, have the same subject line (well, one letter
> different), yet they do different things, please either merge them
> together or give them better names.
>
> thanks,
>
>
> Hello,
> Thanks a lot for your feedback.
> Sorry but I have a doubt. Should I send the whole patchset again with required
> changes ? Also I was working on branch first-patchset for this patchset.
You will note that I did take some patches in this series, you have
those emails. So rebase your series on my latest tree and the applied
patches will go away, and you will be left with what I did not take.
Work from them, I can't do anything with a patch I have already
accepted, right? :)
thanks
greg k-h
next prev parent reply other threads:[~2017-09-11 19:34 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
2017-09-10 15:15 ` [PATCH 01/10] staging: rtl8723bs: Add spaces around '=' Harsha Sharma
2017-09-10 19:44 ` [Outreachy kernel] " Julia Lawall
2017-09-10 15:17 ` [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*' Harsha Sharma
2017-09-10 19:36 ` [Outreachy kernel] " Julia Lawall
2017-09-10 17:21 ` [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!=' Harsha Sharma
2017-09-10 20:01 ` [Outreachy kernel] " Julia Lawall
2017-09-10 17:23 ` [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line Harsha Sharma
2017-09-10 20:33 ` [Outreachy kernel] " Julia Lawall
2017-09-10 17:29 ` [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings Harsha Sharma
2017-09-11 11:09 ` [Outreachy kernel] " Julia Lawall
2017-09-11 11:44 ` Harsha Sharma
2017-09-11 12:11 ` Julia Lawall
2017-09-10 17:32 ` [PATCH 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
2017-09-11 11:13 ` [Outreachy kernel] " Julia Lawall
2017-09-11 16:44 ` Greg KH
2017-09-10 17:34 ` [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space Harsha Sharma
2017-09-11 16:46 ` Greg KH
2017-09-11 18:59 ` Harsha Sharma
2017-09-11 19:34 ` Greg KH [this message]
2017-09-10 17:36 ` [PATCH 08/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
2017-09-10 17:39 ` [PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace Harsha Sharma
2017-09-10 20:18 ` [PATCH] spelling.txt: Add "unnecessary" typo variants Joe Perches
2017-09-10 17:41 ` [PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines Harsha Sharma
2017-09-12 10:08 ` [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space Harsha Sharma
2017-09-12 10:22 ` [Outreachy kernel] " Julia Lawall
2017-09-12 10:33 ` [PATCH v3 07/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
2017-09-12 13:35 ` [PATCH v2 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
2017-09-12 20:41 ` Greg KH
2017-10-08 12:44 ` Harsha Sharma
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170911193410.GD23729@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=harshasharmaiitr@gmail.com \
--cc=outreachy-kernel@googlegroups.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.