LKML Archive on lore.kernel.org help / color / mirror / Atom feed
* [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list @ 2019-05-18 21:00 Paul Walmsley 2019-05-19 5:07 ` Joe Perches 0 siblings, 1 reply; 6+ messages in thread From: Paul Walmsley @ 2019-05-18 21:00 UTC (permalink / raw) To: linux-kernel; +Cc: Paul Walmsley, Andrew Morton 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. Signed-off-by: Paul Walmsley <paul.walmsley@sifive.com> Cc: Andrew Morton <akpm@linux-foundation.org> --- scripts/spelling.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/spelling.txt b/scripts/spelling.txt index 86b87332b9e5..5ae83ce31902 100644 --- a/scripts/spelling.txt +++ b/scripts/spelling.txt @@ -1145,7 +1145,6 @@ senarios||scenarios sentivite||sensitive separatly||separately sepcify||specify -sepc||spec seperated||separated seperately||separately seperate||separate -- 2.20.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list 2019-05-18 21:00 [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list Paul Walmsley @ 2019-05-19 5:07 ` Joe Perches 2019-05-19 18:24 ` Paul Walmsley 0 siblings, 1 reply; 6+ messages in thread From: Joe Perches @ 2019-05-19 5:07 UTC (permalink / raw) To: Paul Walmsley, linux-kernel; +Cc: Andrew Morton 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/net/wireless/realtek/rtlwifi/wifi.h:/* Ref: 802.11i sepc D10.0 7.3.2.25.1 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list 2019-05-19 5:07 ` Joe Perches @ 2019-05-19 18:24 ` Paul Walmsley 2019-05-22 0:14 ` Andrew Morton 0 siblings, 1 reply; 6+ messages in thread From: Paul Walmsley @ 2019-05-19 18:24 UTC (permalink / raw) To: Joe Perches; +Cc: linux-kernel, Andrew Morton 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. - Paul ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list 2019-05-19 18:24 ` Paul Walmsley @ 2019-05-22 0:14 ` Andrew Morton 2019-05-22 4:47 ` Paul Walmsley 0 siblings, 1 reply; 6+ messages in thread From: Andrew Morton @ 2019-05-22 0:14 UTC (permalink / raw) To: Paul Walmsley; +Cc: Joe Perches, linux-kernel 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> --- arch/powerpc/kvm/book3s_xics.c | 2 +- arch/unicore32/include/mach/regs-gpio.h | 2 +- drivers/net/wireless/realtek/rtlwifi/wifi.h | 2 +- drivers/scsi/lpfc/lpfc_init.c | 2 +- drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) --- a/arch/powerpc/kvm/book3s_xics.c~scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix +++ a/arch/powerpc/kvm/book3s_xics.c @@ -830,7 +830,7 @@ static noinline int kvmppc_h_eoi(struct * * Note: If EOI is incorrectly used by SW to lower the CPPR * value (ie more favored), we do not check for rejection of - * a pending interrupt, this is a SW error and PAPR sepcifies + * a pending interrupt, this is a SW error and PAPR specifies * that we don't have to deal with it. * * The sending of an EOI to the ICS is handled after the --- a/arch/unicore32/include/mach/regs-gpio.h~scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix +++ a/arch/unicore32/include/mach/regs-gpio.h @@ -32,7 +32,7 @@ */ #define GPIO_GEDR (PKUNITY_GPIO_BASE + 0x0018) /* - * Sepcial Voltage Detect Reg GPIO_GPIR. + * Special Voltage Detect Reg GPIO_GPIR. */ #define GPIO_GPIR (PKUNITY_GPIO_BASE + 0x0020) --- a/drivers/net/wireless/realtek/rtlwifi/wifi.h~scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix +++ a/drivers/net/wireless/realtek/rtlwifi/wifi.h @@ -597,7 +597,7 @@ enum ht_channel_width { HT_CHANNEL_WIDTH_MAX, }; -/* Ref: 802.11i sepc D10.0 7.3.2.25.1 +/* Ref: 802.11i spec D10.0 7.3.2.25.1 * Cipher Suites Encryption Algorithms */ enum rt_enc_alg { --- a/drivers/scsi/lpfc/lpfc_init.c~scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix +++ a/drivers/scsi/lpfc/lpfc_init.c @@ -2963,7 +2963,7 @@ lpfc_stop_hba_timers(struct lpfc_hba *ph del_timer_sync(&phba->fcp_poll_timer); break; case LPFC_PCI_DEV_OC: - /* Stop any OneConnect device sepcific driver timers */ + /* Stop any OneConnect device specific driver timers */ lpfc_sli4_stop_fcf_redisc_wait_timer(phba); break; default: --- a/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c~scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix +++ a/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c @@ -45,7 +45,7 @@ static u32 phy_CalculateBitShift(u32 Bit /** * Function: PHY_QueryBBReg * -* OverView: Read "sepcific bits" from BB register +* OverView: Read "specific bits" from BB register * * Input: * struct adapter * Adapter, _ ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list 2019-05-22 0:14 ` Andrew Morton @ 2019-05-22 4:47 ` Paul Walmsley 2019-05-22 5:34 ` Joe Perches 0 siblings, 1 reply; 6+ messages in thread From: Paul Walmsley @ 2019-05-22 4:47 UTC (permalink / raw) To: Andrew Morton; +Cc: Joe Perches, linux-kernel 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list 2019-05-22 4:47 ` Paul Walmsley @ 2019-05-22 5:34 ` Joe Perches 0 siblings, 0 replies; 6+ messages in thread From: Joe Perches @ 2019-05-22 5:34 UTC (permalink / raw) To: Paul Walmsley, Andrew Morton; +Cc: linux-kernel On Tue, 2019-05-21 at 21:47 -0700, Paul Walmsley wrote: > 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. I don't believe there is any tail wagging occurring here. There is no code 'fix' in the original proposed patch. It is, as described, effectively a subsystem specific static analysis false positive avoidance patch. And the static analysis tool's false positive report is not active by default. Any scripts/spelling.txt change like a sepc removal could be overridden by using checkpatch's --codespell option. btw: I don't generally add acked-by or reviewed-by to patches as I rather agree with Ted's position on these headers. https://lore.kernel.org/lkml/20190521171618.GD2591@mit.edu/ > I will try to do better next time, Thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-05-22 5:34 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-05-18 21:00 [PATCH] scripts/spelling.txt: drop "sepc" from the misspelling list Paul Walmsley 2019-05-19 5:07 ` Joe Perches 2019-05-19 18:24 ` Paul Walmsley 2019-05-22 0:14 ` Andrew Morton 2019-05-22 4:47 ` Paul Walmsley 2019-05-22 5:34 ` Joe Perches
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).