Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751868AbdFNMTq (ORCPT ); Wed, 14 Jun 2017 08:19:46 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:42772 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750756AbdFNMTp (ORCPT ); Wed, 14 Jun 2017 08:19:45 -0400 Date: Wed, 14 Jun 2017 15:19:08 +0300 From: Dan Carpenter To: Ian W MORRISON Cc: Fabian Wolff , =?iso-8859-1?B?TeF06SBIb3J24XRo?= , devel@driverdev.osuosl.org, linux-kernel@i4.cs.fau.de, Greg KH , linux-kernel@vger.kernel.org, Hans de Goede Subject: Re: [PATCH 1/7] staging: rtl8723bs: wifi_regd.c: put spaces around binary operators Message-ID: <20170614121908.dcemi5scxlmgbpo4@mwanda> References: <1497264376-18840-1-git-send-email-fabian.wolff@fau.de> <1497264376-18840-2-git-send-email-fabian.wolff@fau.de> <20170613143607.h4fzq3otjg45zhr3@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1605 Lines: 32 On Wed, Jun 14, 2017 at 09:49:10PM +1000, Ian W MORRISON wrote: > On 14 June 2017 at 00:36, Dan Carpenter wrote: > > Kernel style is to have spaces around the operators. This is staging > > code so we do all the style fixes. We don't always update older drivers > > but sometimes we do. No one is planning to change those drivers though > > so I guess the answer is no we're not going to update those unless you > > are? > > > > Thanks for the explanation. I assume submitting changes for the > drivers I identified would only be seen as minor corrections to 'the > chaff' resulting in unnecessary churn. If however it is expected that > corrections should be made when identified then I'm willing to prepare > a patch set. I'm happy to take advice either way. I would just leave the old drivers as-is. Having spaces around operators has always been kernel style, but it's only fairly recently that checkpatch.pl started to complain. We keep making checkpatch.pl more stict as time goes on. I think that's good because some reviewers will make you redo patches for style issues so having checkpatch.pl complain early on means you don't have redo the patch. But it also means that old code will never be checkpatch.pl clean because we keep adding new checkpatch warnings. And it's fine that old code has checkpatch warnings. The point of checkpatch is to check new patches not to churn through old code. As a reviewer, I find that checkpatch saves my time because I can often tell people to run it instead of listing all the style complaints. regards, dan carpenter