-----Original Message----- From: Greg KH [mailto:gregkh@xxxxxxxxxxxxxxxxxxx] Sent: Thursday, November 06, 2014 9:11 PM To: Sharma, Sanjeev Cc: Larry.Finger@xxxxxxxxxxxx; Jes.Sorensen@xxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxx; linux-wireless@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx 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@xxxxxxxxxx> > --- > 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-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
