[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <FDC088D3B5555644AE135ED28A7ABDE950E8B8F8@EU-MBX-02.mgc.mentorg.com>
Date: Fri, 7 Nov 2014 11:56:35 +0000
From: "Sharma, Sanjeev" <Sanjeev_Sharma@...tor.com>
To: Greg KH <gregkh@...uxfoundation.org>
CC: "Larry.Finger@...inger.net" <Larry.Finger@...inger.net>,
"Jes.Sorensen@...hat.com" <Jes.Sorensen@...hat.com>,
"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
"linux-wireless@...r.kernel.org" <linux-wireless@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] staging:rtl8723au: core: Fix error reported by
checkpatch.
-----Original Message-----
From: Greg KH [mailto:gregkh@...uxfoundation.org]
Sent: Thursday, November 06, 2014 9:11 PM
To: Sharma, Sanjeev
Cc: Larry.Finger@...inger.net; Jes.Sorensen@...hat.com; devel@...verdev.osuosl.org; linux-wireless@...r.kernel.org; linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging:rtl8723au: core: Fix error reported by checkpatch.
Please specify the "error" in the subject in some way.
On Thu, Nov 06, 2014 at 11:46:13AM +0530, Sanjeev Sharma wrote:
> This is a patch to the rtw_cmd.c file that fixes following error.
>
> ERROR: spaces required around that '>' (ctx:WxV)
> ERROR: that open brace { should be on the previous line
>
> Signed-off-by: Sanjeev Sharma <Sanjeev_Sharma@...tor.com>
> ---
> drivers/staging/rtl8723au/core/rtw_cmd.c | 83
> +++++++++++++++-----------------
> 1 file changed, 40 insertions(+), 43 deletions(-)
This does two different things, please make it different patches.
For individual Error, is Separate patch needed. If Error has been mentioned in description. then it should be OK IMO.
And you sent 2 patches, with no hint as to which one comes before which.
Please resend all of them, correctly numbered in a series, so that I have a chance to get the order correct when applying them.
--
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