Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Tue, 21 May 2019 21:47:26 -0700 (PDT)
From:   Paul Walmsley <paul.walmsley@sifive.com>
To:     Andrew Morton <akpm@linux-foundation.org>
cc:     Joe Perches <joe@perches.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling
 list
In-Reply-To: <20190521171422.c7ef965e39b27f6142788412@linux-foundation.org>
Message-ID: <alpine.DEB.2.21.9999.1905212134310.23235@viisi.sifive.com>
References: <20190518210037.13674-1-paul.walmsley@sifive.com> <201b9ab622b8359225f3a3b673a05047ffce5744.camel@perches.com> <alpine.DEB.2.21.9999.1905191108180.10723@viisi.sifive.com> <20190521171422.c7ef965e39b27f6142788412@linux-foundation.org>
User-Agent: Alpine 2.21.9999 (DEB 301 2018-08-15)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, 21 May 2019, Andrew Morton wrote:

> On Sun, 19 May 2019 11:24:22 -0700 (PDT) Paul Walmsley <paul.walmsley@sifive.com> wrote:
> 
> > On Sat, 18 May 2019, Joe Perches wrote:
> > 
> > > On Sat, 2019-05-18 at 14:00 -0700, Paul Walmsley wrote:
> > > > The RISC-V architecture has a register named the "Supervisor Exception
> > > > Program Counter", or "sepc".  This abbreviation triggers
> > > > checkpatch.pl's misspelling detector, resulting in noise in the
> > > > checkpatch output.  The risk that this noise could cause more useful
> > > > warnings to be missed seems to outweigh the harm of an occasional
> > > > misspelling of "spec".  Thus drop the "sepc" entry from the
> > > > misspelling list.
> > > 
> > > I would agree if you first fixed the existing sepc/spec
> > > and sepcific/specific typos.
> > > 
> > > arch/powerpc/kvm/book3s_xics.c:	 * a pending interrupt, this is a SW error and PAPR sepcifies
> > > arch/unicore32/include/mach/regs-gpio.h: * Sepcial Voltage Detect Reg GPIO_GPIR.
> > > drivers/scsi/lpfc/lpfc_init.c:		/* Stop any OneConnect device sepcific driver timers */
> > > drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c:* OverView:	Read "sepcific bits" from BB register
> > > drivers/net/wireless/realtek/rtlwifi/wifi.h:/* Ref: 802.11i sepc D10.0 7.3.2.25.1
> > 
> > Your agreement shouldn't be needed for the patch I sent.
> 
> I always find Joe's input to be very useful.
> 
> Here:
> 
> From: Andrew Morton <akpm@linux-foundation.org>
> Subject: scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix
> 
> fix existing "sepc" instances, per Joe
> 
> Cc: Joe Perches <joe@perches.com>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

Thanks Andrew.  Sorry that you had to do it.

Reviewed-by: Paul Walmsley <paul.walmsley@sifive.com>

What troubled me about Joe's message is that it seems like poor kernel 
developer precedent to block a fix for static analysis false positives to 
fix comment spelling errors -- particularly considering that four out of 
five of them were unrelated to the actual patch in question.  While 
comment spelling fixes are worthwhile, I think we should make sure that 
the "tail doesn't wag the dog" by prioritizing code fixes first.

Reflecting on it on Sunday evening, if Joe had acked the patch, or added a 
Reviewed-by, and asked whether I might send a patch to fix those spelling 
errors, it probably would have gotten done.  

I will try to do better next time,


- Paul