Linux-Renesas-SoC Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
@ 2019-05-14 15:33 Chris Paterson
  2019-05-14 21:19 ` Stephen Boyd
  0 siblings, 1 reply; 8+ messages in thread
From: Chris Paterson @ 2019-05-14 15:33 UTC (permalink / raw)
  To: Andrew Morton, Geert Uytterhoeven
  Cc: Biju Das, linux-clk, linux-renesas-soc, linux-kernel, Chris Paterson

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(-)

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
-- 
2.17.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-14 15:33 [PATCH] scripts/spelling.txt: Add spelling fix for prohibited Chris Paterson
@ 2019-05-14 21:19 ` Stephen Boyd
  2019-05-15  7:32   ` Geert Uytterhoeven
  0 siblings, 1 reply; 8+ messages in thread
From: Stephen Boyd @ 2019-05-14 21:19 UTC (permalink / raw)
  To: Andrew Morton, Chris Paterson, Geert Uytterhoeven
  Cc: Biju Das, linux-clk, linux-renesas-soc, linux-kernel, Chris Paterson

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>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-14 21:19 ` Stephen Boyd
@ 2019-05-15  7:32   ` Geert Uytterhoeven
  2019-05-16 21:54     ` Stephen Boyd
  0 siblings, 1 reply; 8+ messages in thread
From: Geert Uytterhoeven @ 2019-05-15  7:32 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Andrew Morton, Chris Paterson, Geert Uytterhoeven, Biju Das,
	linux-clk, Linux-Renesas, Linux Kernel Mailing List

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

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-15  7:32   ` Geert Uytterhoeven
@ 2019-05-16 21:54     ` Stephen Boyd
  2019-05-21  8:57       ` Geert Uytterhoeven
  0 siblings, 1 reply; 8+ messages in thread
From: Stephen Boyd @ 2019-05-16 21:54 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Andrew Morton, Chris Paterson, Geert Uytterhoeven, Biju Das,
	linux-clk, Linux-Renesas, Linux Kernel Mailing List

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.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-16 21:54     ` Stephen Boyd
@ 2019-05-21  8:57       ` Geert Uytterhoeven
  2019-05-21 18:01         ` Stephen Boyd
  0 siblings, 1 reply; 8+ messages in thread
From: Geert Uytterhoeven @ 2019-05-21  8:57 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Andrew Morton, Chris Paterson, Geert Uytterhoeven, Biju Das,
	linux-clk, Linux-Renesas, Linux Kernel Mailing List

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

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-21  8:57       ` Geert Uytterhoeven
@ 2019-05-21 18:01         ` Stephen Boyd
  2019-05-23  9:07           ` Chris Paterson
  0 siblings, 1 reply; 8+ messages in thread
From: Stephen Boyd @ 2019-05-21 18:01 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Andrew Morton, Chris Paterson, Geert Uytterhoeven, Biju Das,
	linux-clk, Linux-Renesas, Linux Kernel Mailing List

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.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-21 18:01         ` Stephen Boyd
@ 2019-05-23  9:07           ` Chris Paterson
  2019-05-23 18:06             ` Stephen Boyd
  0 siblings, 1 reply; 8+ messages in thread
From: Chris Paterson @ 2019-05-23  9:07 UTC (permalink / raw)
  To: Stephen Boyd, Geert Uytterhoeven
  Cc: Andrew Morton, Geert Uytterhoeven, Biju Das, linux-clk,
	Linux-Renesas, Linux Kernel Mailing List

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] scripts/spelling.txt: Add spelling fix for prohibited
  2019-05-23  9:07           ` Chris Paterson
@ 2019-05-23 18:06             ` Stephen Boyd
  0 siblings, 0 replies; 8+ messages in thread
From: Stephen Boyd @ 2019-05-23 18:06 UTC (permalink / raw)
  To: Chris Paterson, Geert Uytterhoeven
  Cc: Andrew Morton, Geert Uytterhoeven, Biju Das, linux-clk,
	Linux-Renesas, Linux Kernel Mailing List

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.


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, back to index

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-14 15:33 [PATCH] scripts/spelling.txt: Add spelling fix for prohibited Chris Paterson
2019-05-14 21:19 ` Stephen Boyd
2019-05-15  7:32   ` Geert Uytterhoeven
2019-05-16 21:54     ` Stephen Boyd
2019-05-21  8:57       ` Geert Uytterhoeven
2019-05-21 18:01         ` Stephen Boyd
2019-05-23  9:07           ` Chris Paterson
2019-05-23 18:06             ` Stephen Boyd

Linux-Renesas-SoC Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-renesas-soc/0 linux-renesas-soc/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-renesas-soc linux-renesas-soc/ https://lore.kernel.org/linux-renesas-soc \
		linux-renesas-soc@vger.kernel.org linux-renesas-soc@archiver.kernel.org
	public-inbox-index linux-renesas-soc


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-renesas-soc


AGPL code for this site: git clone https://public-inbox.org/ public-inbox