Re: [PATCH] clk: renesas: cpg-mssr: Rename vsp1-(sy|rt) clocks to vsp(s|r)

2020-09-03 Thread Geert Uytterhoeven
On Mon, Aug 31, 2020 at 8:37 PM Lad Prabhakar
 wrote:
> VSP1 instances VSPS (which stands for "VSP Standard") and VSPR (which
> stands for "VSP for Resizing") were wrongly named as "vsp1-sy" and
> "vsp1-rt". The clock section in the SoC datasheets misunderstood the
> abbreviations as meaning VSP System and VSP Realtime, and named the
> corresponding clocks VSP1(SY) and VSP1(RT). This mistake has been
> carried over to the kernel code.
>
> This patch fixes this by renaming the clock names to "vsps" and "vspr".
>
> Inspired from commit 79ea9934b8df ("ARM: shmobile: r8a7790: Rename
> VSP1_(SY|RT) clocks to VSP1_(S|R)")
>
> Signed-off-by: Lad Prabhakar 

Reviewed-by: Geert Uytterhoeven 
i.e. will queue in renesas-devel for v5.10.

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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


[PATCH] clk: renesas: cpg-mssr: Rename vsp1-(sy|rt) clocks to vsp(s|r)

2020-08-31 Thread Lad Prabhakar
VSP1 instances VSPS (which stands for "VSP Standard") and VSPR (which
stands for "VSP for Resizing") were wrongly named as "vsp1-sy" and
"vsp1-rt". The clock section in the SoC datasheets misunderstood the
abbreviations as meaning VSP System and VSP Realtime, and named the
corresponding clocks VSP1(SY) and VSP1(RT). This mistake has been
carried over to the kernel code.

This patch fixes this by renaming the clock names to "vsps" and "vspr".

Inspired from commit 79ea9934b8df ("ARM: shmobile: r8a7790: Rename
VSP1_(SY|RT) clocks to VSP1_(S|R)")

Signed-off-by: Lad Prabhakar 
---
 drivers/clk/renesas/r8a7743-cpg-mssr.c  | 2 +-
 drivers/clk/renesas/r8a7745-cpg-mssr.c  | 2 +-
 drivers/clk/renesas/r8a77470-cpg-mssr.c | 2 +-
 drivers/clk/renesas/r8a7790-cpg-mssr.c  | 4 ++--
 drivers/clk/renesas/r8a7791-cpg-mssr.c  | 2 +-
 drivers/clk/renesas/r8a7792-cpg-mssr.c  | 2 +-
 drivers/clk/renesas/r8a7794-cpg-mssr.c  | 2 +-
 7 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/clk/renesas/r8a7743-cpg-mssr.c 
b/drivers/clk/renesas/r8a7743-cpg-mssr.c
index c01d9af2525a..0bba12a48d22 100644
--- a/drivers/clk/renesas/r8a7743-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7743-cpg-mssr.c
@@ -92,7 +92,7 @@ static const struct mssr_mod_clk r8a7743_mod_clks[] 
__initconst = {
DEF_MOD("tmu0",  125,   R8A7743_CLK_CP),
DEF_MOD("vsp1du1",   127,   R8A7743_CLK_ZS),
DEF_MOD("vsp1du0",   128,   R8A7743_CLK_ZS),
-   DEF_MOD("vsp1-sy",   131,   R8A7743_CLK_ZS),
+   DEF_MOD("vsps",  131,   R8A7743_CLK_ZS),
DEF_MOD("scifa2",202,   R8A7743_CLK_MP),
DEF_MOD("scifa1",203,   R8A7743_CLK_MP),
DEF_MOD("scifa0",204,   R8A7743_CLK_MP),
diff --git a/drivers/clk/renesas/r8a7745-cpg-mssr.c 
b/drivers/clk/renesas/r8a7745-cpg-mssr.c
index 493874e5ebee..dc4a64e8dfb5 100644
--- a/drivers/clk/renesas/r8a7745-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7745-cpg-mssr.c
@@ -90,7 +90,7 @@ static const struct mssr_mod_clk r8a7745_mod_clks[] 
__initconst = {
DEF_MOD("cmt0",  124,   R8A7745_CLK_R),
DEF_MOD("tmu0",  125,   R8A7745_CLK_CP),
DEF_MOD("vsp1du0",   128,   R8A7745_CLK_ZS),
-   DEF_MOD("vsp1-sy",   131,   R8A7745_CLK_ZS),
+   DEF_MOD("vsps",  131,   R8A7745_CLK_ZS),
DEF_MOD("scifa2",202,   R8A7745_CLK_MP),
DEF_MOD("scifa1",203,   R8A7745_CLK_MP),
DEF_MOD("scifa0",204,   R8A7745_CLK_MP),
diff --git a/drivers/clk/renesas/r8a77470-cpg-mssr.c 
b/drivers/clk/renesas/r8a77470-cpg-mssr.c
index d81ae65f0d18..f3d6e65011d7 100644
--- a/drivers/clk/renesas/r8a77470-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a77470-cpg-mssr.c
@@ -85,7 +85,7 @@ static const struct mssr_mod_clk r8a77470_mod_clks[] 
__initconst = {
DEF_MOD("tmu2",  122,   R8A77470_CLK_P),
DEF_MOD("cmt0",  124,   R8A77470_CLK_R),
DEF_MOD("vsp1du0",   128,   R8A77470_CLK_ZS),
-   DEF_MOD("vsp1-sy",   131,   R8A77470_CLK_ZS),
+   DEF_MOD("vsps",  131,   R8A77470_CLK_ZS),
DEF_MOD("msiof2",205,   R8A77470_CLK_MP),
DEF_MOD("msiof1",208,   R8A77470_CLK_MP),
DEF_MOD("sys-dmac1", 218,   R8A77470_CLK_ZS),
diff --git a/drivers/clk/renesas/r8a7790-cpg-mssr.c 
b/drivers/clk/renesas/r8a7790-cpg-mssr.c
index c57cb93f8315..f7d233e0c142 100644
--- a/drivers/clk/renesas/r8a7790-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7790-cpg-mssr.c
@@ -108,8 +108,8 @@ static const struct mssr_mod_clk r8a7790_mod_clks[] 
__initconst = {
DEF_MOD("tmu0",  125,   R8A7790_CLK_CP),
DEF_MOD("vsp1du1",   127,   R8A7790_CLK_ZS),
DEF_MOD("vsp1du0",   128,   R8A7790_CLK_ZS),
-   DEF_MOD("vsp1-rt",   130,   R8A7790_CLK_ZS),
-   DEF_MOD("vsp1-sy",   131,   R8A7790_CLK_ZS),
+   DEF_MOD("vspr",  130,   R8A7790_CLK_ZS),
+   DEF_MOD("vsps",  131,   R8A7790_CLK_ZS),
DEF_MOD("scifa2",202,   R8A7790_CLK_MP),
DEF_MOD("scifa1",203,   R8A7790_CLK_MP),
DEF_MOD("scifa0",204,   R8A7790_CLK_MP),
diff --git a/drivers/clk/renesas/r8a7791-cpg-mssr.c 
b/drivers/clk/renesas/r8a7791-cpg-mssr.c
index 65702debcabb..a0de784868da 100644
--- a/drivers/clk/renesas/r8a7791-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7791-cpg-mssr.c
@@ -102,7 +102,7 @@ static const struct mssr_mod_clk r8a7791_mod_clks[] 
__initconst = {
DEF_MOD("tmu0",  125,   R8A7791_CLK_CP),
DEF_MOD("vsp1du1",   127,   R8A7791_CLK_ZS),
DEF_MOD("vsp1du0",   128,   R8A7791_CLK_ZS),
-   DEF_MOD("vsp1-sy",   131,