Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753379AbdFMNME (ORCPT ); Tue, 13 Jun 2017 09:12:04 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:53180 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752456AbdFMNMC (ORCPT ); Tue, 13 Jun 2017 09:12:02 -0400 Date: Tue, 13 Jun 2017 15:11:49 +0200 From: Greg KH To: Fabian Wolff Cc: horvatmate@gmail.com, hdegoede@redhat.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, linux-kernel@i4.cs.fau.de Subject: Re: [PATCH 1/7] staging: rtl8723bs: wifi_regd.c: put spaces around binary operators Message-ID: <20170613131149.GA23757@kroah.com> References: <1497264376-18840-1-git-send-email-fabian.wolff@fau.de> <1497264376-18840-2-git-send-email-fabian.wolff@fau.de> <20170613125843.GE14290@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170613125843.GE14290@kroah.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2101 Lines: 54 On Tue, Jun 13, 2017 at 02:58:43PM +0200, Greg KH wrote: > On Mon, Jun 12, 2017 at 12:46:10PM +0200, Fabian Wolff wrote: > > This patch adds spaces around the binary operators '-' and '+', which > > is the preferred style, and reformats a pointer argument declaration > > to fix a false positive "spaces preferred around that '*'" message > > reported by checkpatch.pl. > > > > Signed-off-by: Fabian Wolff > > Signed-off-by: Mate Horvath > > --- > > drivers/staging/rtl8723bs/os_dep/wifi_regd.c | 10 +++++----- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/drivers/staging/rtl8723bs/os_dep/wifi_regd.c b/drivers/staging/rtl8723bs/os_dep/wifi_regd.c > > index 9c61125..803eab0 100644 > > --- a/drivers/staging/rtl8723bs/os_dep/wifi_regd.c > > +++ b/drivers/staging/rtl8723bs/os_dep/wifi_regd.c > > @@ -20,7 +20,7 @@ > > > > /* 2G chan 01 - chan 11 */ > > #define RTW_2GHZ_CH01_11 \ > > - REG_RULE(2412-10, 2462+10, 40, 0, 20, 0) > > + REG_RULE(2412 - 10, 2462 + 10, 40, 0, 20, 0) > > > > /* > > *We enable active scan on these a case > > @@ -29,12 +29,12 @@ > > > > /* 2G chan 12 - chan 13, PASSIV SCAN */ > > #define RTW_2GHZ_CH12_13 \ > > - REG_RULE(2467-10, 2472+10, 40, 0, 20, \ > > + REG_RULE(2467 - 10, 2472 + 10, 40, 0, 20, \ > > NL80211_RRF_PASSIVE_SCAN) > > > > /* 2G chan 14, PASSIVS SCAN, NO OFDM (B only) */ > > #define RTW_2GHZ_CH14 \ > > - REG_RULE(2484-10, 2484+10, 40, 0, 20, \ > > + REG_RULE(2484 - 10, 2484 + 10, 40, 0, 20, \ > > NL80211_RRF_PASSIVE_SCAN | NL80211_RRF_NO_OFDM) > > > > static const struct ieee80211_regdomain rtw_regdom_rd = { > > @@ -116,8 +116,8 @@ static int _rtw_reg_notifier_apply(struct wiphy *wiphy, > > } > > > > static const struct ieee80211_regdomain *_rtw_regdomain_select(struct > > - rtw_regulatory > > - *reg) > > + rtw_regulatory * > > + reg) > > Why make this last change? It's not an arithmatic one :( And can you refresh this series and resend the remaining ones? They all didn't apply :(