Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753376AbdFMM7C (ORCPT ); Tue, 13 Jun 2017 08:59:02 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:51298 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752480AbdFMM7B (ORCPT ); Tue, 13 Jun 2017 08:59:01 -0400 Date: Tue, 13 Jun 2017 14:58:43 +0200 From: Greg KH To: Fabian Wolff Cc: linux-kernel@i4.cs.fau.de, horvatmate@gmail.com, hdegoede@redhat.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/7] staging: rtl8723bs: wifi_regd.c: put spaces around binary operators Message-ID: <20170613125843.GE14290@kroah.com> References: <1497264376-18840-1-git-send-email-fabian.wolff@fau.de> <1497264376-18840-2-git-send-email-fabian.wolff@fau.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1497264376-18840-2-git-send-email-fabian.wolff@fau.de> 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: 1854 Lines: 50 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 :(