* [PATCH] clk: samsung: s3c: Remove unneeded enumeration [not found] <CGME20171127023145epcas1p1b97c3c66ce624f5362c8d9bf74997c7f@epcas1p1.samsung.com> @ 2017-11-27 2:31 ` Chanwoo Choi 2017-12-20 9:14 ` Chanwoo Choi 2018-01-03 17:29 ` Sylwester Nawrocki 0 siblings, 2 replies; 4+ messages in thread From: Chanwoo Choi @ 2017-11-27 2:31 UTC (permalink / raw) To: linux-arm-kernel This patch just removes the unneeded enumeration for PLL index. Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> --- drivers/clk/samsung/clk-s3c2412.c | 11 ++--------- drivers/clk/samsung/clk-s3c2443.c | 17 ++++------------- drivers/clk/samsung/clk-s3c64xx.c | 17 ++++++----------- 3 files changed, 12 insertions(+), 33 deletions(-) diff --git a/drivers/clk/samsung/clk-s3c2412.c b/drivers/clk/samsung/clk-s3c2412.c index b8340a49921b..1555e407529e 100644 --- a/drivers/clk/samsung/clk-s3c2412.c +++ b/drivers/clk/samsung/clk-s3c2412.c @@ -27,11 +27,6 @@ #define CLKSRC 0x1c #define SWRST 0x30 -/* list of PLLs to be registered */ -enum s3c2412_plls { - mpll, upll, -}; - static void __iomem *reg_base; #ifdef CONFIG_PM_SLEEP @@ -144,10 +139,8 @@ struct samsung_mux_clock s3c2412_muxes[] __initdata = { }; static struct samsung_pll_clock s3c2412_plls[] __initdata = { - [mpll] = PLL(pll_s3c2440_mpll, MPLL, "mpll", "xti", - LOCKTIME, MPLLCON, NULL), - [upll] = PLL(pll_s3c2410_upll, UPLL, "upll", "urefclk", - LOCKTIME, UPLLCON, NULL), + PLL(pll_s3c2440_mpll, MPLL, "mpll", "xti", LOCKTIME, MPLLCON, NULL), + PLL(pll_s3c2410_upll, UPLL, "upll", "urefclk", LOCKTIME, UPLLCON, NULL), }; struct samsung_gate_clock s3c2412_gates[] __initdata = { diff --git a/drivers/clk/samsung/clk-s3c2443.c b/drivers/clk/samsung/clk-s3c2443.c index d94b85a42356..9580a6baf4d7 100644 --- a/drivers/clk/samsung/clk-s3c2443.c +++ b/drivers/clk/samsung/clk-s3c2443.c @@ -41,11 +41,6 @@ enum supported_socs { S3C2450, }; -/* list of PLLs to be registered */ -enum s3c2443_plls { - mpll, epll, -}; - static void __iomem *reg_base; #ifdef CONFIG_PM_SLEEP @@ -225,10 +220,8 @@ struct samsung_clock_alias s3c2443_common_aliases[] __initdata = { /* S3C2416 specific clocks */ static struct samsung_pll_clock s3c2416_pll_clks[] __initdata = { - [mpll] = PLL(pll_6552_s3c2416, MPLL, "mpll", "mpllref", - LOCKCON0, MPLLCON, NULL), - [epll] = PLL(pll_6553, EPLL, "epll", "epllref", - LOCKCON1, EPLLCON, NULL), + PLL(pll_6552_s3c2416, MPLL, "mpll", "mpllref", LOCKCON0, MPLLCON, NULL), + PLL(pll_6553, EPLL, "epll", "epllref", LOCKCON1, EPLLCON, NULL), }; PNAME(s3c2416_hsmmc0_p) = { "sclk_hsmmc0", "sclk_hsmmcext" }; @@ -279,10 +272,8 @@ struct samsung_clock_alias s3c2416_aliases[] __initdata = { /* S3C2443 specific clocks */ static struct samsung_pll_clock s3c2443_pll_clks[] __initdata = { - [mpll] = PLL(pll_3000, MPLL, "mpll", "mpllref", - LOCKCON0, MPLLCON, NULL), - [epll] = PLL(pll_2126, EPLL, "epll", "epllref", - LOCKCON1, EPLLCON, NULL), + PLL(pll_3000, MPLL, "mpll", "mpllref", LOCKCON0, MPLLCON, NULL), + PLL(pll_2126, EPLL, "epll", "epllref", LOCKCON1, EPLLCON, NULL), }; static struct clk_div_table armdiv_s3c2443_d[] = { diff --git a/drivers/clk/samsung/clk-s3c64xx.c b/drivers/clk/samsung/clk-s3c64xx.c index 7306867a0ab8..6db01cf5ab83 100644 --- a/drivers/clk/samsung/clk-s3c64xx.c +++ b/drivers/clk/samsung/clk-s3c64xx.c @@ -56,11 +56,6 @@ #define GATE_ON(_id, cname, pname, o, b) \ GATE(_id, cname, pname, o, b, CLK_IGNORE_UNUSED, 0) -/* list of PLLs to be registered */ -enum s3c64xx_plls { - apll, mpll, epll, -}; - static void __iomem *reg_base; static bool is_s3c6400; @@ -364,12 +359,12 @@ static void __init s3c64xx_clk_sleep_init(void) {} /* List of PLL clocks. */ static struct samsung_pll_clock s3c64xx_pll_clks[] __initdata = { - [apll] = PLL(pll_6552, FOUT_APLL, "fout_apll", "fin_pll", - APLL_LOCK, APLL_CON, NULL), - [mpll] = PLL(pll_6552, FOUT_MPLL, "fout_mpll", "fin_pll", - MPLL_LOCK, MPLL_CON, NULL), - [epll] = PLL(pll_6553, FOUT_EPLL, "fout_epll", "fin_pll", - EPLL_LOCK, EPLL_CON0, NULL), + PLL(pll_6552, FOUT_APLL, "fout_apll", "fin_pll", + APLL_LOCK, APLL_CON, NULL), + PLL(pll_6552, FOUT_MPLL, "fout_mpll", "fin_pll", + MPLL_LOCK, MPLL_CON, NULL), + PLL(pll_6553, FOUT_EPLL, "fout_epll", "fin_pll", + EPLL_LOCK, EPLL_CON0, NULL), }; /* Aliases for common s3c64xx clocks. */ -- 1.9.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] clk: samsung: s3c: Remove unneeded enumeration 2017-11-27 2:31 ` [PATCH] clk: samsung: s3c: Remove unneeded enumeration Chanwoo Choi @ 2017-12-20 9:14 ` Chanwoo Choi 2018-01-03 17:29 ` Sylwester Nawrocki 1 sibling, 0 replies; 4+ messages in thread From: Chanwoo Choi @ 2017-12-20 9:14 UTC (permalink / raw) To: linux-arm-kernel Dear Sylwester, Gently Ping. Regards, Chanwoo Choi On 2017? 11? 27? 11:31, Chanwoo Choi wrote: > This patch just removes the unneeded enumeration for PLL index. > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > --- > drivers/clk/samsung/clk-s3c2412.c | 11 ++--------- > drivers/clk/samsung/clk-s3c2443.c | 17 ++++------------- > drivers/clk/samsung/clk-s3c64xx.c | 17 ++++++----------- > 3 files changed, 12 insertions(+), 33 deletions(-) > > diff --git a/drivers/clk/samsung/clk-s3c2412.c b/drivers/clk/samsung/clk-s3c2412.c > index b8340a49921b..1555e407529e 100644 > --- a/drivers/clk/samsung/clk-s3c2412.c > +++ b/drivers/clk/samsung/clk-s3c2412.c > @@ -27,11 +27,6 @@ > #define CLKSRC 0x1c > #define SWRST 0x30 > > -/* list of PLLs to be registered */ > -enum s3c2412_plls { > - mpll, upll, > -}; > - > static void __iomem *reg_base; > > #ifdef CONFIG_PM_SLEEP > @@ -144,10 +139,8 @@ struct samsung_mux_clock s3c2412_muxes[] __initdata = { > }; > > static struct samsung_pll_clock s3c2412_plls[] __initdata = { > - [mpll] = PLL(pll_s3c2440_mpll, MPLL, "mpll", "xti", > - LOCKTIME, MPLLCON, NULL), > - [upll] = PLL(pll_s3c2410_upll, UPLL, "upll", "urefclk", > - LOCKTIME, UPLLCON, NULL), > + PLL(pll_s3c2440_mpll, MPLL, "mpll", "xti", LOCKTIME, MPLLCON, NULL), > + PLL(pll_s3c2410_upll, UPLL, "upll", "urefclk", LOCKTIME, UPLLCON, NULL), > }; > > struct samsung_gate_clock s3c2412_gates[] __initdata = { > diff --git a/drivers/clk/samsung/clk-s3c2443.c b/drivers/clk/samsung/clk-s3c2443.c > index d94b85a42356..9580a6baf4d7 100644 > --- a/drivers/clk/samsung/clk-s3c2443.c > +++ b/drivers/clk/samsung/clk-s3c2443.c > @@ -41,11 +41,6 @@ enum supported_socs { > S3C2450, > }; > > -/* list of PLLs to be registered */ > -enum s3c2443_plls { > - mpll, epll, > -}; > - > static void __iomem *reg_base; > > #ifdef CONFIG_PM_SLEEP > @@ -225,10 +220,8 @@ struct samsung_clock_alias s3c2443_common_aliases[] __initdata = { > /* S3C2416 specific clocks */ > > static struct samsung_pll_clock s3c2416_pll_clks[] __initdata = { > - [mpll] = PLL(pll_6552_s3c2416, MPLL, "mpll", "mpllref", > - LOCKCON0, MPLLCON, NULL), > - [epll] = PLL(pll_6553, EPLL, "epll", "epllref", > - LOCKCON1, EPLLCON, NULL), > + PLL(pll_6552_s3c2416, MPLL, "mpll", "mpllref", LOCKCON0, MPLLCON, NULL), > + PLL(pll_6553, EPLL, "epll", "epllref", LOCKCON1, EPLLCON, NULL), > }; > > PNAME(s3c2416_hsmmc0_p) = { "sclk_hsmmc0", "sclk_hsmmcext" }; > @@ -279,10 +272,8 @@ struct samsung_clock_alias s3c2416_aliases[] __initdata = { > /* S3C2443 specific clocks */ > > static struct samsung_pll_clock s3c2443_pll_clks[] __initdata = { > - [mpll] = PLL(pll_3000, MPLL, "mpll", "mpllref", > - LOCKCON0, MPLLCON, NULL), > - [epll] = PLL(pll_2126, EPLL, "epll", "epllref", > - LOCKCON1, EPLLCON, NULL), > + PLL(pll_3000, MPLL, "mpll", "mpllref", LOCKCON0, MPLLCON, NULL), > + PLL(pll_2126, EPLL, "epll", "epllref", LOCKCON1, EPLLCON, NULL), > }; > > static struct clk_div_table armdiv_s3c2443_d[] = { > diff --git a/drivers/clk/samsung/clk-s3c64xx.c b/drivers/clk/samsung/clk-s3c64xx.c > index 7306867a0ab8..6db01cf5ab83 100644 > --- a/drivers/clk/samsung/clk-s3c64xx.c > +++ b/drivers/clk/samsung/clk-s3c64xx.c > @@ -56,11 +56,6 @@ > #define GATE_ON(_id, cname, pname, o, b) \ > GATE(_id, cname, pname, o, b, CLK_IGNORE_UNUSED, 0) > > -/* list of PLLs to be registered */ > -enum s3c64xx_plls { > - apll, mpll, epll, > -}; > - > static void __iomem *reg_base; > static bool is_s3c6400; > > @@ -364,12 +359,12 @@ static void __init s3c64xx_clk_sleep_init(void) {} > > /* List of PLL clocks. */ > static struct samsung_pll_clock s3c64xx_pll_clks[] __initdata = { > - [apll] = PLL(pll_6552, FOUT_APLL, "fout_apll", "fin_pll", > - APLL_LOCK, APLL_CON, NULL), > - [mpll] = PLL(pll_6552, FOUT_MPLL, "fout_mpll", "fin_pll", > - MPLL_LOCK, MPLL_CON, NULL), > - [epll] = PLL(pll_6553, FOUT_EPLL, "fout_epll", "fin_pll", > - EPLL_LOCK, EPLL_CON0, NULL), > + PLL(pll_6552, FOUT_APLL, "fout_apll", "fin_pll", > + APLL_LOCK, APLL_CON, NULL), > + PLL(pll_6552, FOUT_MPLL, "fout_mpll", "fin_pll", > + MPLL_LOCK, MPLL_CON, NULL), > + PLL(pll_6553, FOUT_EPLL, "fout_epll", "fin_pll", > + EPLL_LOCK, EPLL_CON0, NULL), > }; > > /* Aliases for common s3c64xx clocks. */ > ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] clk: samsung: s3c: Remove unneeded enumeration 2017-11-27 2:31 ` [PATCH] clk: samsung: s3c: Remove unneeded enumeration Chanwoo Choi 2017-12-20 9:14 ` Chanwoo Choi @ 2018-01-03 17:29 ` Sylwester Nawrocki 2018-01-03 22:58 ` Chanwoo Choi 1 sibling, 1 reply; 4+ messages in thread From: Sylwester Nawrocki @ 2018-01-03 17:29 UTC (permalink / raw) To: linux-arm-kernel On 11/27/2017 03:31 AM, Chanwoo Choi wrote: > This patch just removes the unneeded enumeration for PLL index. > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> Thanks for the patch Chanwoo, I have applied it to my tree but I'm afraid it will now need to wait until v4.17. Stephen, if you decide to apply it directly for v4.16-rc1 Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com> -- Thanks, Sylwester ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] clk: samsung: s3c: Remove unneeded enumeration 2018-01-03 17:29 ` Sylwester Nawrocki @ 2018-01-03 22:58 ` Chanwoo Choi 0 siblings, 0 replies; 4+ messages in thread From: Chanwoo Choi @ 2018-01-03 22:58 UTC (permalink / raw) To: linux-arm-kernel On 2018? 01? 04? 02:29, Sylwester Nawrocki wrote: > On 11/27/2017 03:31 AM, Chanwoo Choi wrote: >> This patch just removes the unneeded enumeration for PLL index. >> >> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > > > Thanks for the patch Chanwoo, I have applied it to my tree but > I'm afraid it will now need to wait until v4.17. No problem. Thanks. > > Stephen, if you decide to apply it directly for v4.16-rc1 > > Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com> > > -- > Thanks, > Sylwester > > -- Best Regards, Chanwoo Choi Samsung Electronics ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-01-03 22:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20171127023145epcas1p1b97c3c66ce624f5362c8d9bf74997c7f@epcas1p1.samsung.com>
2017-11-27 2:31 ` [PATCH] clk: samsung: s3c: Remove unneeded enumeration Chanwoo Choi
2017-12-20 9:14 ` Chanwoo Choi
2018-01-03 17:29 ` Sylwester Nawrocki
2018-01-03 22:58 ` Chanwoo Choi
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).