| Message ID | 20190514153341.22540-1-chris.paterson2@renesas.com (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Kieran Bingham |
| Headers | show |
| Series | scripts/spelling.txt: Add spelling fix for prohibited | expand |
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>
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
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.
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
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.
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
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.
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
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(-)