Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

Path	csiph.com!aioe.org!bofh.it!news.nic.it!robomod
From	Greg Donald <gdonald@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors
Date	Mon, 22 Jun 2015 13:30:01 +0200
Message-ID	<pE85j-2e7-7@gated-at.bofh.it> (permalink)
References	<pDy1Q-XJ-11@gated-at.bofh.it> <pE7VE-22X-15@gated-at.bofh.it>
X-Original-To	Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=G91JJVRrYG6bJP6yf/8y4rRfY6vpx3RgWrkGK1+XG0E=; b=064MYB/oAhbUthK9QPKe5MkG5MhFbHQp7ItVrQ/kP22S8UCfAO3/oQSOLI4IkedRoz PY5j0Ud1mXO/o46JirRyZRPQZAGBK4bqgn5qov7TQ92rTUvf9F0TOtf+DHCWLzuvCh4l 6zcJPfTKIHIg7VI35sPgsD4sKuy0yO20tNs394DPDROMIU4K8DKrW5sO0UYMKAbQMKxw PKdvC03ACvwOD7f+3MTmCIsh9ld0KK5695+BJpJBTdH/LLVhWC5nn4gwKV7Mcj84uQo1 0QXROTike2oX+7IvT4LsIdg3LM/rdfxbbqXTuGooWpqUPsX771P4keYCwmNnPkdB5umE pvpg==
X-Received	by 10.50.79.169 with SMTP id k9mr13368103igx.44.1434972330707; Mon, 22 Jun 2015 04:25:30 -0700 (PDT)
MIME-Version	1.0
Content-Type	text/plain; charset=UTF-8
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	47
Organization	linux.* mail to news gateway
X-Original-Cc	Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Aya Mahfouz <mahfouz.saif.elyazal@gmail.com>, Haneen Mohammed <hamohammed.sa@gmail.com>, Cristina Opriceana <cristina.opriceana@gmail.com>, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
X-Original-Date	Mon, 22 Jun 2015 06:25:11 -0500
X-Original-Message-ID	<CAO+WgCZFGiD1REh+mkPgQmK0OecAaa42m+LENNAU3L3mD8fxLg@mail.gmail.com>
X-Original-References	<1434833768-8336-1-git-send-email-gdonald@gmail.com> <20150622111014.GB3355@sudip-PC>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	aioe.org linux.kernel:1169846

Show key headers only | View raw

Umm.. you have to fix more than one error if there's more than one
error on or very near the same line you are already fixing, else
checkpatch.pl complains that your patch has errors.  Not to mention
Greg KH has never complained:

http://git.kernel.org/cgit/linux/kernel/git/gregkh/staging.git/log/?qt=grep&q=gdonald



On Mon, Jun 22, 2015 at 6:10 AM, Sudip Mukherjee
<sudipm.mukherjee@gmail.com> wrote:
> On Sat, Jun 20, 2015 at 03:56:08PM -0500, Greg Donald wrote:
>> Fix checkpatch.pl "space required before the open parenthesis '('" errors
>>
>> Signed-off-by: Greg Donald <gdonald@gmail.com>
>> ---
>>  .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c    | 97 ++++++++++------------
>>  1 file changed, 44 insertions(+), 53 deletions(-)
>>
>> diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
>> index c2588f8..df20979 100644
>> --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
>> +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
> <snip>
>> @@ -127,8 +127,7 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString )
>>       static u8       EWC11NHTCap[] = {0x00, 0x90, 0x4c, 0x33};       // For 11n EWC definition, 2007.07.17, by Emily
>>       PHT_CAPABILITY_ELE              pCapELE;
>>
>> -     if(!memcmp(CapIE, EWC11NHTCap, sizeof(EWC11NHTCap)))
>> -     {
>> +     if (!memcmp(CapIE, EWC11NHTCap, sizeof(EWC11NHTCap))) {
>                                                           ^^^^^^^
> this becomes a different kind of change than what is mentioned in your
> commit message.
> You are also adding space after ',' later in your patch.
> please do only one kind of change in one patch.
>
> regards
> sudip



-- 
Greg Donald
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before  the open parenthesis '('" errors Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-06-22 13:20 +0200
  Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before  the open parenthesis '('" errors Greg Donald <gdonald@gmail.com> - 2015-06-22 13:30 +0200

csiph-web