Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <4e42f2580d8b473ac7e25642b039adf73b5efe06.camel@perches.com>
Subject: Re: [PATCH v3] checkpatch: add a rule to check general block
 comment style
From:   Joe Perches <joe@perches.com>
To:     Ani Sinha <ani@anisinha.ca>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>
Cc:     Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        anirban.sinha@nokia.com, mikelley@microsoft.com,
        Andy Whitcroft <apw@canonical.com>,
        Dwaipayan Ray <dwaipayanray1@gmail.com>
Date:   Mon, 19 Jul 2021 00:52:17 -0700
In-Reply-To: <alpine.DEB.2.22.394.2107191221570.12107@anisinha-lenovo>
References: <20210714063422.2164699-1-ani@anisinha.ca>
         <alpine.DEB.2.22.394.2107162145100.3277407@anisinha-lenovo>
         <CAKXUXMzxyQxYKdnuVbZnUFbYFuKMq+Vc5LxSR-_fKtH2dv-4wA@mail.gmail.com>
         <alpine.DEB.2.21.2107181853320.55905@ani-ubuntu>
         <c32293fc03a8673ade348ed4451c60dfdf9bb2c1.camel@perches.com>
         <alpine.DEB.2.22.394.2107191054260.9400@anisinha-lenovo>
         <CAKXUXMyJ9Ab_Fvpk8WAimEnDjcOhJRe2uODkC=RxCgsN8T0WyQ@mail.gmail.com>
         <alpine.DEB.2.22.394.2107191221570.12107@anisinha-lenovo>
Content-Type: text/plain; charset="ISO-8859-1"
User-Agent: Evolution 3.40.0-1 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Mon, 2021-07-19 at 12:25 +0530, Ani Sinha wrote:

> I do not see why we cannot add this rule to checkpatch. If the
> reviewer likes the other style of commenting they can always ask for a
> correction. Having checkpatch agree with Linus' preferred style of
> commenting and the preferred documeted style of commenting (which seems to
> be the same) does make everything uniform and agreeable.

Too many novice developers take checkpatch output as dicta.

It's not.

It's just produces suggestions that should _always_ be taken
not very seriously.  Those suggestions should perhaps be
considered, but good taste should always override a brainless
script.

_Very_ few senior developers really care that much about any
particular comment style.

These are the same senior developers that would be burdened
with unnecessary patches to review from those novice developers
that believe checkpatch should always be followed.

Do not unnecessarily burden senior developers.
They are generally have other priorities.