scripts/spelling.txt: Add spelling fix for prohibited
diff mbox series

Message ID 20190514153341.22540-1-chris.paterson2@renesas.com
State Under Review
Delegated to: Kieran Bingham
Headers show
Series
  • scripts/spelling.txt: Add spelling fix for prohibited
Related show

Commit Message

Chris Paterson May 14, 2019, 3:33 p.m. UTC
Misspelling 'prohibited' is quite common in the real world, although
surprisingly not so much in the Linux Kernel. In addition to fixing the
typo we may as well add it to the spelling checker.

Also adding the present participle (prohibiting).

Fixes: 5bf2fbbef50c ("clk: renesas: cpg-mssr: Add r8a77470 support")

Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
---
 drivers/clk/renesas/r8a77470-cpg-mssr.c | 2 +-
 scripts/spelling.txt                    | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

Comments

Stephen Boyd May 14, 2019, 9:19 p.m. UTC | #1
Quoting Chris Paterson (2019-05-14 08:33:41)
> Misspelling 'prohibited' is quite common in the real world, although
> surprisingly not so much in the Linux Kernel. In addition to fixing the
> typo we may as well add it to the spelling checker.
> 
> Also adding the present participle (prohibiting).
> 
> Fixes: 5bf2fbbef50c ("clk: renesas: cpg-mssr: Add r8a77470 support")
> 
> Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
> ---

Acked-by: Stephen Boyd <sboyd@kernel.org>
Geert Uytterhoeven May 15, 2019, 7:32 a.m. UTC | #2
On Tue, May 14, 2019 at 11:19 PM Stephen Boyd <sboyd@kernel.org> wrote:
> Quoting Chris Paterson (2019-05-14 08:33:41)
> > Misspelling 'prohibited' is quite common in the real world, although
> > surprisingly not so much in the Linux Kernel. In addition to fixing the
> > typo we may as well add it to the spelling checker.
> >
> > Also adding the present participle (prohibiting).
> >
> > Fixes: 5bf2fbbef50c ("clk: renesas: cpg-mssr: Add r8a77470 support")
> >
> > Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

> Acked-by: Stephen Boyd <sboyd@kernel.org>

Thanks!

So I guess I'll queue this in clk-renesas-for-v5.3?

Gr{oetje,eeting}s,

                        Geert
Stephen Boyd May 16, 2019, 9:54 p.m. UTC | #3
Quoting Geert Uytterhoeven (2019-05-15 00:32:46)
> On Tue, May 14, 2019 at 11:19 PM Stephen Boyd <sboyd@kernel.org> wrote:
> > Quoting Chris Paterson (2019-05-14 08:33:41)
> > > Misspelling 'prohibited' is quite common in the real world, although
> > > surprisingly not so much in the Linux Kernel. In addition to fixing the
> > > typo we may as well add it to the spelling checker.
> > >
> > > Also adding the present participle (prohibiting).
> > >
> > > Fixes: 5bf2fbbef50c ("clk: renesas: cpg-mssr: Add r8a77470 support")
> > >
> > > Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> > Acked-by: Stephen Boyd <sboyd@kernel.org>
> 
> Thanks!
> 
> So I guess I'll queue this in clk-renesas-for-v5.3?
> 

Guess so! Or Andrew does it.
Geert Uytterhoeven May 21, 2019, 8:57 a.m. UTC | #4
Hi Stephen, Andrew,

On Thu, May 16, 2019 at 11:54 PM Stephen Boyd <sboyd@kernel.org> wrote:
> Quoting Geert Uytterhoeven (2019-05-15 00:32:46)
> > On Tue, May 14, 2019 at 11:19 PM Stephen Boyd <sboyd@kernel.org> wrote:
> > > Quoting Chris Paterson (2019-05-14 08:33:41)
> > > > Misspelling 'prohibited' is quite common in the real world, although
> > > > surprisingly not so much in the Linux Kernel. In addition to fixing the
> > > > typo we may as well add it to the spelling checker.
> > > >
> > > > Also adding the present participle (prohibiting).
> > > >
> > > > Fixes: 5bf2fbbef50c ("clk: renesas: cpg-mssr: Add r8a77470 support")
> > > >
> > > > Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
> >
> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> > > Acked-by: Stephen Boyd <sboyd@kernel.org>
> >
> > Thanks!
> >
> > So I guess I'll queue this in clk-renesas-for-v5.3?
> >
>
> Guess so! Or Andrew does it.

Given the change to scripts/spelling.txt conflicts with b937856a5db2cb7a
("scripts/spelling.txt: add more typos to spelling.txt and sort") in
linux-next, and the risk of conflict with future changes to
drivers/clk/renesas/r8a77470-cpg-mssr.c is slim, I'd like to defer this
to Andrew.

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert
Stephen Boyd May 21, 2019, 6:01 p.m. UTC | #5
Quoting Geert Uytterhoeven (2019-05-21 01:57:20)
> Hi Stephen, Andrew,
> 
> On Thu, May 16, 2019 at 11:54 PM Stephen Boyd <sboyd@kernel.org> wrote:
> >
> > Guess so! Or Andrew does it.
> 
> Given the change to scripts/spelling.txt conflicts with b937856a5db2cb7a
> ("scripts/spelling.txt: add more typos to spelling.txt and sort") in
> linux-next, and the risk of conflict with future changes to
> drivers/clk/renesas/r8a77470-cpg-mssr.c is slim, I'd like to defer this
> to Andrew.
> 
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

Ok. Or Chris can split the patch into the clk part and the scripts/
part?  They're only loosely related to each other, so I'm not sure why
they were sent in one patch besides for ease of transport.
Chris Paterson May 23, 2019, 9:07 a.m. UTC | #6
Hello Stephen,

> From: Stephen Boyd <sboyd@kernel.org>
> Sent: 21 May 2019 19:02
> 
> Quoting Geert Uytterhoeven (2019-05-21 01:57:20)
> > Hi Stephen, Andrew,
> >
> > On Thu, May 16, 2019 at 11:54 PM Stephen Boyd <sboyd@kernel.org>
> wrote:
> > >
> > > Guess so! Or Andrew does it.
> >
> > Given the change to scripts/spelling.txt conflicts with b937856a5db2cb7a
> > ("scripts/spelling.txt: add more typos to spelling.txt and sort") in
> > linux-next, and the risk of conflict with future changes to
> > drivers/clk/renesas/r8a77470-cpg-mssr.c is slim, I'd like to defer this
> > to Andrew.
> >
> > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Ok. Or Chris can split the patch into the clk part and the scripts/
> part?  They're only loosely related to each other, so I'm not sure why
> they were sent in one patch besides for ease of transport.

Happy to split the patch if you want. I combined the patch as I saw that done in a couple of examples in the git log.

However, I got an automated email from akpm saying that this patch was added to the -mm tree; so I don't know if you still want me to split up this patch or not.
Let me know.

Kind regards, Chris
Stephen Boyd May 23, 2019, 6:06 p.m. UTC | #7
Quoting Chris Paterson (2019-05-23 02:07:51)
> > From: Stephen Boyd <sboyd@kernel.org>
> > 
> > Ok. Or Chris can split the patch into the clk part and the scripts/
> > part?  They're only loosely related to each other, so I'm not sure why
> > they were sent in one patch besides for ease of transport.
> 
> Happy to split the patch if you want. I combined the patch as I saw that done in a couple of examples in the git log.
> 
> However, I got an automated email from akpm saying that this patch was added to the -mm tree; so I don't know if you still want me to split up this patch or not.
> Let me know.
> 

Everything still looks fine to me so I dont' think you need to do
anything.

Patch
diff mbox series

diff --git a/drivers/clk/renesas/r8a77470-cpg-mssr.c b/drivers/clk/renesas/r8a77470-cpg-mssr.c
index ab0fb10b6bf0..d81ae65f0d18 100644
--- a/drivers/clk/renesas/r8a77470-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a77470-cpg-mssr.c
@@ -175,7 +175,7 @@  static const unsigned int r8a77470_crit_mod_clks[] __initconst = {
  *---------------------------------------------------
  * 0  0		20		x80	x78	x50
  * 0  1		26		x60	x60	x56
- * 1  0		Prohibitted setting
+ * 1  0		Prohibited setting
  * 1  1		30		x52	x52	x50
  *
  * *1 :	Table 7.4 indicates VCO output (PLL0 = VCO)
diff --git a/scripts/spelling.txt b/scripts/spelling.txt
index dcd25a31f9f5..135459f59db6 100644
--- a/scripts/spelling.txt
+++ b/scripts/spelling.txt
@@ -1310,6 +1310,8 @@  programmble||programmable
 programm||program
 programms||programs
 progresss||progress
+prohibitted||prohibited
+prohibitting||prohibiting
 promiscouos||promiscuous
 promiscous||promiscuous
 promixity||proximity