* [RFC PATCH 0/2] ARM: OMAP4+: Clk registration update @ 2012-10-18 16:38 ` Grygorii Strashko 0 siblings, 0 replies; 12+ messages in thread From: Grygorii Strashko @ 2012-10-18 16:38 UTC (permalink / raw) To: Paul Walmsley Cc: Mike Turquette, Kevin Hilman, Tony Lindgren, linux-omap, linux-arm-kernel, Grygorii Strashko Currently we use CK_446X etc to differentiate which clk needs to be registered on which SoC. The following patch series removes that requirement, instead SoC and SoC variants are detected on the fly and only the required clocks are registered in the system. This will help to remove the CK_XYZ macros from the system. Please comment if this is an acceptable approach - at a first glance, it seems to pave way to be capable of being replaced potentially by DT data as well. Grygorii Strashko (2): ARM: OMAP2+: clock: Add omap2_clks_register API ARM: OMAP4: clock: use omap4_clks_register API arch/arm/mach-omap2/clock.c | 36 ++++++++++++++++++++++++++++++ arch/arm/mach-omap2/clock.h | 3 +++ arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- 3 files changed, 60 insertions(+), 19 deletions(-) -- 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [RFC PATCH 0/2] ARM: OMAP4+: Clk registration update @ 2012-10-18 16:38 ` Grygorii Strashko 0 siblings, 0 replies; 12+ messages in thread From: Grygorii Strashko @ 2012-10-18 16:38 UTC (permalink / raw) To: linux-arm-kernel Currently we use CK_446X etc to differentiate which clk needs to be registered on which SoC. The following patch series removes that requirement, instead SoC and SoC variants are detected on the fly and only the required clocks are registered in the system. This will help to remove the CK_XYZ macros from the system. Please comment if this is an acceptable approach - at a first glance, it seems to pave way to be capable of being replaced potentially by DT data as well. Grygorii Strashko (2): ARM: OMAP2+: clock: Add omap2_clks_register API ARM: OMAP4: clock: use omap4_clks_register API arch/arm/mach-omap2/clock.c | 36 ++++++++++++++++++++++++++++++ arch/arm/mach-omap2/clock.h | 3 +++ arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- 3 files changed, 60 insertions(+), 19 deletions(-) -- 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [RFC PATCH 1/2] ARM: OMAP2+: clock: Add omap2_clks_register API 2012-10-18 16:38 ` Grygorii Strashko @ 2012-10-18 16:38 ` Grygorii Strashko -1 siblings, 0 replies; 12+ messages in thread From: Grygorii Strashko @ 2012-10-18 16:38 UTC (permalink / raw) To: Paul Walmsley Cc: Mike Turquette, Kevin Hilman, Tony Lindgren, linux-omap, linux-arm-kernel, Grygorii Strashko Now the cpu_mask is used to differentiate clocks per each OMAP platform, SoC version or revision. Such approach has few disadvantages: - the specific CK_XXX flag need to be added and maintained for each OMAP SoC; - it's difficult to update clock tree data in case of differences between OMAP SoC revisions. In addition, there are duplicated code in each clockXXX_data.c files, so it seems, reasonable to remove it: - call clk_preinit() for each clock; - call clkdev_add(); clk_register(); omap2_init_clk_clkdm(); for each clock; Hence, add omap4_clks_register() API which would allow to register and init few set of clocks and to organize clocks data in the following way (for example for OMAP4): - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 SoCs - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x SoCs - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x SoCs Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> --- arch/arm/mach-omap2/clock.c | 36 ++++++++++++++++++++++++++++++++++++ arch/arm/mach-omap2/clock.h | 3 +++ 2 files changed, 39 insertions(+) diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c index 961ac8f..d84f174 100644 --- a/arch/arm/mach-omap2/clock.c +++ b/arch/arm/mach-omap2/clock.c @@ -521,3 +521,39 @@ struct clk_functions omap2_clk_functions = { .clk_disable_unused = omap2_clk_disable_unused, }; +/** + * omap_clks_register() - register set of clocks + * @clks: array of clock definitions + * + * The last array item should contain NULL value in c->lk.clk. + */ +int __init omap2_clks_register(struct omap_clk *clks) +{ + struct omap_clk *c; + int r; + + if (!clks) + return -EINVAL; + + c = clks; + while (c->lk.clk) { + clk_preinit(c->lk.clk); + c++; + } + + c = clks; + while (c->lk.clk) { + clkdev_add(&c->lk); + r = clk_register(c->lk.clk); + if (r < 0) { + pr_warn("%s: Clock register failure %d.\n", + c->lk.clk->name, r); + } + + omap2_init_clk_clkdm(c->lk.clk); + + c++; + } + + return 0; +} diff --git a/arch/arm/mach-omap2/clock.h b/arch/arm/mach-omap2/clock.h index 35ec5f3..920f3f2 100644 --- a/arch/arm/mach-omap2/clock.h +++ b/arch/arm/mach-omap2/clock.h @@ -19,6 +19,7 @@ #include <linux/kernel.h> #include <plat/clock.h> +#include <plat/clkdev_omap.h> /* CM_CLKSEL2_PLL.CORE_CLK_SRC bits (2XXX) */ #define CORE_CLK_SRC_32K 0x0 @@ -132,6 +133,8 @@ void omap2_clk_print_new_rates(const char *hfclkin_ck_name, const char *core_ck_name, const char *mpu_ck_name); +int omap2_clks_register(struct omap_clk *clks); + extern u16 cpu_mask; extern const struct clkops clkops_omap2_dflt_wait; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RFC PATCH 1/2] ARM: OMAP2+: clock: Add omap2_clks_register API @ 2012-10-18 16:38 ` Grygorii Strashko 0 siblings, 0 replies; 12+ messages in thread From: Grygorii Strashko @ 2012-10-18 16:38 UTC (permalink / raw) To: linux-arm-kernel Now the cpu_mask is used to differentiate clocks per each OMAP platform, SoC version or revision. Such approach has few disadvantages: - the specific CK_XXX flag need to be added and maintained for each OMAP SoC; - it's difficult to update clock tree data in case of differences between OMAP SoC revisions. In addition, there are duplicated code in each clockXXX_data.c files, so it seems, reasonable to remove it: - call clk_preinit() for each clock; - call clkdev_add(); clk_register(); omap2_init_clk_clkdm(); for each clock; Hence, add omap4_clks_register() API which would allow to register and init few set of clocks and to organize clocks data in the following way (for example for OMAP4): - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 SoCs - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x SoCs - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x SoCs Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> --- arch/arm/mach-omap2/clock.c | 36 ++++++++++++++++++++++++++++++++++++ arch/arm/mach-omap2/clock.h | 3 +++ 2 files changed, 39 insertions(+) diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c index 961ac8f..d84f174 100644 --- a/arch/arm/mach-omap2/clock.c +++ b/arch/arm/mach-omap2/clock.c @@ -521,3 +521,39 @@ struct clk_functions omap2_clk_functions = { .clk_disable_unused = omap2_clk_disable_unused, }; +/** + * omap_clks_register() - register set of clocks + * @clks: array of clock definitions + * + * The last array item should contain NULL value in c->lk.clk. + */ +int __init omap2_clks_register(struct omap_clk *clks) +{ + struct omap_clk *c; + int r; + + if (!clks) + return -EINVAL; + + c = clks; + while (c->lk.clk) { + clk_preinit(c->lk.clk); + c++; + } + + c = clks; + while (c->lk.clk) { + clkdev_add(&c->lk); + r = clk_register(c->lk.clk); + if (r < 0) { + pr_warn("%s: Clock register failure %d.\n", + c->lk.clk->name, r); + } + + omap2_init_clk_clkdm(c->lk.clk); + + c++; + } + + return 0; +} diff --git a/arch/arm/mach-omap2/clock.h b/arch/arm/mach-omap2/clock.h index 35ec5f3..920f3f2 100644 --- a/arch/arm/mach-omap2/clock.h +++ b/arch/arm/mach-omap2/clock.h @@ -19,6 +19,7 @@ #include <linux/kernel.h> #include <plat/clock.h> +#include <plat/clkdev_omap.h> /* CM_CLKSEL2_PLL.CORE_CLK_SRC bits (2XXX) */ #define CORE_CLK_SRC_32K 0x0 @@ -132,6 +133,8 @@ void omap2_clk_print_new_rates(const char *hfclkin_ck_name, const char *core_ck_name, const char *mpu_ck_name); +int omap2_clks_register(struct omap_clk *clks); + extern u16 cpu_mask; extern const struct clkops clkops_omap2_dflt_wait; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API 2012-10-18 16:38 ` Grygorii Strashko @ 2012-10-18 16:38 ` Grygorii Strashko -1 siblings, 0 replies; 12+ messages in thread From: Grygorii Strashko @ 2012-10-18 16:38 UTC (permalink / raw) To: Paul Walmsley Cc: Mike Turquette, Kevin Hilman, Tony Lindgren, linux-omap, linux-arm-kernel, Grygorii Strashko Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks array and add corresponding set of clocks per each SoC: - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x and don't rely on platform specific flags any more. Use omap4_clks_register() API to register and init OMAP4 set of clocks. With this change, we can now safely remove CK_443X/CK_446X etc macro usage. It has not been done in this patch, but if the approach is OK, then, it is possible to do the same. One additional benefit seen is that the match logic can entirely be skipped. Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> --- arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- 1 file changed, 21 insertions(+), 19 deletions(-) diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c index 6efc30c..4060c9c 100644 --- a/arch/arm/mach-omap2/clock44xx_data.c +++ b/arch/arm/mach-omap2/clock44xx_data.c @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), CLK(NULL, "aess_fck", &aess_fck, CK_443X), - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), + CLK(NULL, NULL, NULL, CK_443X), +}; + +static struct omap_clk omap443x_clks[] = { + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), + CLK(NULL, NULL, NULL, CK_443X), +}; + + +static struct omap_clk omap446x_clks[] = { + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), + CLK(NULL, NULL, NULL, CK_446X), }; int __init omap4xxx_clk_init(void) { - struct omap_clk *c; - u32 cpu_clkflg; - if (cpu_is_omap443x()) { cpu_mask = RATE_IN_4430; - cpu_clkflg = CK_443X; } else if (cpu_is_omap446x() || cpu_is_omap447x()) { cpu_mask = RATE_IN_4460 | RATE_IN_4430; - cpu_clkflg = CK_446X | CK_443X; if (cpu_is_omap447x()) pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) * omap2_clk_disable_clkdm_control(); */ - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); - c++) - clk_preinit(c->lk.clk); - - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); - c++) - if (c->cpu & cpu_clkflg) { - clkdev_add(&c->lk); - clk_register(c->lk.clk); - omap2_init_clk_clkdm(c->lk.clk); - } + omap2_clks_register(omap44xx_clks); + + if (cpu_is_omap443x()) + omap2_clks_register(omap443x_clks); + else if (cpu_is_omap446x() || cpu_is_omap447x()) + omap2_clks_register(omap446x_clks); + else + return 0; /* Disable autoidle on all clocks; let the PM code enable it later */ omap_clk_disable_autoidle_all(); -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API @ 2012-10-18 16:38 ` Grygorii Strashko 0 siblings, 0 replies; 12+ messages in thread From: Grygorii Strashko @ 2012-10-18 16:38 UTC (permalink / raw) To: linux-arm-kernel Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks array and add corresponding set of clocks per each SoC: - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x and don't rely on platform specific flags any more. Use omap4_clks_register() API to register and init OMAP4 set of clocks. With this change, we can now safely remove CK_443X/CK_446X etc macro usage. It has not been done in this patch, but if the approach is OK, then, it is possible to do the same. One additional benefit seen is that the match logic can entirely be skipped. Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> --- arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- 1 file changed, 21 insertions(+), 19 deletions(-) diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c index 6efc30c..4060c9c 100644 --- a/arch/arm/mach-omap2/clock44xx_data.c +++ b/arch/arm/mach-omap2/clock44xx_data.c @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), CLK(NULL, "aess_fck", &aess_fck, CK_443X), - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), + CLK(NULL, NULL, NULL, CK_443X), +}; + +static struct omap_clk omap443x_clks[] = { + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), + CLK(NULL, NULL, NULL, CK_443X), +}; + + +static struct omap_clk omap446x_clks[] = { + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), + CLK(NULL, NULL, NULL, CK_446X), }; int __init omap4xxx_clk_init(void) { - struct omap_clk *c; - u32 cpu_clkflg; - if (cpu_is_omap443x()) { cpu_mask = RATE_IN_4430; - cpu_clkflg = CK_443X; } else if (cpu_is_omap446x() || cpu_is_omap447x()) { cpu_mask = RATE_IN_4460 | RATE_IN_4430; - cpu_clkflg = CK_446X | CK_443X; if (cpu_is_omap447x()) pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) * omap2_clk_disable_clkdm_control(); */ - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); - c++) - clk_preinit(c->lk.clk); - - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); - c++) - if (c->cpu & cpu_clkflg) { - clkdev_add(&c->lk); - clk_register(c->lk.clk); - omap2_init_clk_clkdm(c->lk.clk); - } + omap2_clks_register(omap44xx_clks); + + if (cpu_is_omap443x()) + omap2_clks_register(omap443x_clks); + else if (cpu_is_omap446x() || cpu_is_omap447x()) + omap2_clks_register(omap446x_clks); + else + return 0; /* Disable autoidle on all clocks; let the PM code enable it later */ omap_clk_disable_autoidle_all(); -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API 2012-10-18 16:38 ` Grygorii Strashko @ 2012-10-18 17:53 ` Mike Turquette -1 siblings, 0 replies; 12+ messages in thread From: Mike Turquette @ 2012-10-18 17:53 UTC (permalink / raw) To: Paul Walmsley Cc: Kevin Hilman, Tony Lindgren, linux-omap, linux-arm-kernel, Grygorii Strashko Quoting Grygorii Strashko (2012-10-18 09:38:17) > Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks > array and add corresponding set of clocks per each SoC: > - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 > - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x > - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x > and don't rely on platform specific flags any more. > > Use omap4_clks_register() API to register and init OMAP4 set of > clocks. > > With this change, we can now safely remove CK_443X/CK_446X etc macro > usage. It has not been done in this patch, but if the approach is OK, > then, it is possible to do the same. > In addition to removing CK_443X/CK_446X you could also get rid of struct omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and the CLK macro by changing the clk array to be of type struct clk_lookup and using the CLKDEV_INIT macro. > One additional benefit seen is that the match logic can entirely be > skipped. > I wonder if splitting the tables would make initdata any easier. It would be nice to get more functional changes out of this series alongside the data reorganization. Also have you looked at splitting the tables for OMAP2/3? Regards, Mike > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > --- > arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- > 1 file changed, 21 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > index 6efc30c..4060c9c 100644 > --- a/arch/arm/mach-omap2/clock44xx_data.c > +++ b/arch/arm/mach-omap2/clock44xx_data.c > @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { > CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), > CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), > CLK(NULL, "aess_fck", &aess_fck, CK_443X), > - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { > CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > +static struct omap_clk omap443x_clks[] = { > + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > + > +static struct omap_clk omap446x_clks[] = { > + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > + CLK(NULL, NULL, NULL, CK_446X), > }; > > int __init omap4xxx_clk_init(void) > { > - struct omap_clk *c; > - u32 cpu_clkflg; > - > if (cpu_is_omap443x()) { > cpu_mask = RATE_IN_4430; > - cpu_clkflg = CK_443X; > } else if (cpu_is_omap446x() || cpu_is_omap447x()) { > cpu_mask = RATE_IN_4460 | RATE_IN_4430; > - cpu_clkflg = CK_446X | CK_443X; > > if (cpu_is_omap447x()) > pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); > @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) > * omap2_clk_disable_clkdm_control(); > */ > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - clk_preinit(c->lk.clk); > - > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - if (c->cpu & cpu_clkflg) { > - clkdev_add(&c->lk); > - clk_register(c->lk.clk); > - omap2_init_clk_clkdm(c->lk.clk); > - } > + omap2_clks_register(omap44xx_clks); > + > + if (cpu_is_omap443x()) > + omap2_clks_register(omap443x_clks); > + else if (cpu_is_omap446x() || cpu_is_omap447x()) > + omap2_clks_register(omap446x_clks); > + else > + return 0; > > /* Disable autoidle on all clocks; let the PM code enable it later */ > omap_clk_disable_autoidle_all(); > -- > 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API @ 2012-10-18 17:53 ` Mike Turquette 0 siblings, 0 replies; 12+ messages in thread From: Mike Turquette @ 2012-10-18 17:53 UTC (permalink / raw) To: linux-arm-kernel Quoting Grygorii Strashko (2012-10-18 09:38:17) > Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks > array and add corresponding set of clocks per each SoC: > - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 > - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x > - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x > and don't rely on platform specific flags any more. > > Use omap4_clks_register() API to register and init OMAP4 set of > clocks. > > With this change, we can now safely remove CK_443X/CK_446X etc macro > usage. It has not been done in this patch, but if the approach is OK, > then, it is possible to do the same. > In addition to removing CK_443X/CK_446X you could also get rid of struct omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and the CLK macro by changing the clk array to be of type struct clk_lookup and using the CLKDEV_INIT macro. > One additional benefit seen is that the match logic can entirely be > skipped. > I wonder if splitting the tables would make initdata any easier. It would be nice to get more functional changes out of this series alongside the data reorganization. Also have you looked at splitting the tables for OMAP2/3? Regards, Mike > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > --- > arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- > 1 file changed, 21 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > index 6efc30c..4060c9c 100644 > --- a/arch/arm/mach-omap2/clock44xx_data.c > +++ b/arch/arm/mach-omap2/clock44xx_data.c > @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { > CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), > CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), > CLK(NULL, "aess_fck", &aess_fck, CK_443X), > - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { > CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > +static struct omap_clk omap443x_clks[] = { > + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > + > +static struct omap_clk omap446x_clks[] = { > + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > + CLK(NULL, NULL, NULL, CK_446X), > }; > > int __init omap4xxx_clk_init(void) > { > - struct omap_clk *c; > - u32 cpu_clkflg; > - > if (cpu_is_omap443x()) { > cpu_mask = RATE_IN_4430; > - cpu_clkflg = CK_443X; > } else if (cpu_is_omap446x() || cpu_is_omap447x()) { > cpu_mask = RATE_IN_4460 | RATE_IN_4430; > - cpu_clkflg = CK_446X | CK_443X; > > if (cpu_is_omap447x()) > pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); > @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) > * omap2_clk_disable_clkdm_control(); > */ > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - clk_preinit(c->lk.clk); > - > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - if (c->cpu & cpu_clkflg) { > - clkdev_add(&c->lk); > - clk_register(c->lk.clk); > - omap2_init_clk_clkdm(c->lk.clk); > - } > + omap2_clks_register(omap44xx_clks); > + > + if (cpu_is_omap443x()) > + omap2_clks_register(omap443x_clks); > + else if (cpu_is_omap446x() || cpu_is_omap447x()) > + omap2_clks_register(omap446x_clks); > + else > + return 0; > > /* Disable autoidle on all clocks; let the PM code enable it later */ > omap_clk_disable_autoidle_all(); > -- > 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API 2012-10-18 17:53 ` Mike Turquette @ 2012-10-19 10:34 ` Strashko, Grygorii -1 siblings, 0 replies; 12+ messages in thread From: Strashko, Grygorii @ 2012-10-19 10:34 UTC (permalink / raw) To: Mike Turquette, Paul Walmsley Cc: Hilman, Kevin, Tony Lindgren, linux-omap, linux-arm-kernel Hi Mike, > In addition to removing CK_443X/CK_446X you could also get rid of struct > omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and > the CLK macro by changing the clk array to be of type struct clk_lookup > and using the CLKDEV_INIT macro. Yes. It may be done. As I understand, this approach is applicable for you, if so i can continue working on it. right? > I wonder if splitting the tables would make initdata any easier. It OMAP5,4,2 - there are no so much differences between SOcs, so it looks good and simple. OMAP3 - it is a little bit more complex. > would be nice to get more functional changes out of this series > alongside the data reorganization. Could you provide more details, pls? > Also have you looked at splitting the tables for OMAP2/3? Yes. I think, it can be done. Best regards, Grygorii Strashko | GlobalLogic ________________________________________ From: Mike Turquette [mturquette@linaro.org] Sent: Thursday, October 18, 2012 8:53 PM To: Strashko, Grygorii; Paul Walmsley Cc: Hilman, Kevin; Tony Lindgren; linux-omap@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Strashko, Grygorii Subject: Re: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API Quoting Grygorii Strashko (2012-10-18 09:38:17) > Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks > array and add corresponding set of clocks per each SoC: > - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 > - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x > - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x > and don't rely on platform specific flags any more. > > Use omap4_clks_register() API to register and init OMAP4 set of > clocks. > > With this change, we can now safely remove CK_443X/CK_446X etc macro > usage. It has not been done in this patch, but if the approach is OK, > then, it is possible to do the same. > In addition to removing CK_443X/CK_446X you could also get rid of struct omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and the CLK macro by changing the clk array to be of type struct clk_lookup and using the CLKDEV_INIT macro. > One additional benefit seen is that the match logic can entirely be > skipped. > I wonder if splitting the tables would make initdata any easier. It would be nice to get more functional changes out of this series alongside the data reorganization. Also have you looked at splitting the tables for OMAP2/3? Regards, Mike > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > --- > arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- > 1 file changed, 21 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > index 6efc30c..4060c9c 100644 > --- a/arch/arm/mach-omap2/clock44xx_data.c > +++ b/arch/arm/mach-omap2/clock44xx_data.c > @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { > CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), > CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), > CLK(NULL, "aess_fck", &aess_fck, CK_443X), > - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { > CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > +static struct omap_clk omap443x_clks[] = { > + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > + > +static struct omap_clk omap446x_clks[] = { > + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > + CLK(NULL, NULL, NULL, CK_446X), > }; > > int __init omap4xxx_clk_init(void) > { > - struct omap_clk *c; > - u32 cpu_clkflg; > - > if (cpu_is_omap443x()) { > cpu_mask = RATE_IN_4430; > - cpu_clkflg = CK_443X; > } else if (cpu_is_omap446x() || cpu_is_omap447x()) { > cpu_mask = RATE_IN_4460 | RATE_IN_4430; > - cpu_clkflg = CK_446X | CK_443X; > > if (cpu_is_omap447x()) > pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); > @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) > * omap2_clk_disable_clkdm_control(); > */ > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - clk_preinit(c->lk.clk); > - > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - if (c->cpu & cpu_clkflg) { > - clkdev_add(&c->lk); > - clk_register(c->lk.clk); > - omap2_init_clk_clkdm(c->lk.clk); > - } > + omap2_clks_register(omap44xx_clks); > + > + if (cpu_is_omap443x()) > + omap2_clks_register(omap443x_clks); > + else if (cpu_is_omap446x() || cpu_is_omap447x()) > + omap2_clks_register(omap446x_clks); > + else > + return 0; > > /* Disable autoidle on all clocks; let the PM code enable it later */ > omap_clk_disable_autoidle_all(); > -- > 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API @ 2012-10-19 10:34 ` Strashko, Grygorii 0 siblings, 0 replies; 12+ messages in thread From: Strashko, Grygorii @ 2012-10-19 10:34 UTC (permalink / raw) To: linux-arm-kernel Hi Mike, > In addition to removing CK_443X/CK_446X you could also get rid of struct > omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and > the CLK macro by changing the clk array to be of type struct clk_lookup > and using the CLKDEV_INIT macro. Yes. It may be done. As I understand, this approach is applicable for you, if so i can continue working on it. right? > I wonder if splitting the tables would make initdata any easier. It OMAP5,4,2 - there are no so much differences between SOcs, so it looks good and simple. OMAP3 - it is a little bit more complex. > would be nice to get more functional changes out of this series > alongside the data reorganization. Could you provide more details, pls? > Also have you looked at splitting the tables for OMAP2/3? Yes. I think, it can be done. Best regards, Grygorii Strashko | GlobalLogic ________________________________________ From: Mike Turquette [mturquette at linaro.org] Sent: Thursday, October 18, 2012 8:53 PM To: Strashko, Grygorii; Paul Walmsley Cc: Hilman, Kevin; Tony Lindgren; linux-omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org; Strashko, Grygorii Subject: Re: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API Quoting Grygorii Strashko (2012-10-18 09:38:17) > Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks > array and add corresponding set of clocks per each SoC: > - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 > - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x > - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x > and don't rely on platform specific flags any more. > > Use omap4_clks_register() API to register and init OMAP4 set of > clocks. > > With this change, we can now safely remove CK_443X/CK_446X etc macro > usage. It has not been done in this patch, but if the approach is OK, > then, it is possible to do the same. > In addition to removing CK_443X/CK_446X you could also get rid of struct omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and the CLK macro by changing the clk array to be of type struct clk_lookup and using the CLKDEV_INIT macro. > One additional benefit seen is that the match logic can entirely be > skipped. > I wonder if splitting the tables would make initdata any easier. It would be nice to get more functional changes out of this series alongside the data reorganization. Also have you looked at splitting the tables for OMAP2/3? Regards, Mike > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > --- > arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- > 1 file changed, 21 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > index 6efc30c..4060c9c 100644 > --- a/arch/arm/mach-omap2/clock44xx_data.c > +++ b/arch/arm/mach-omap2/clock44xx_data.c > @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { > CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), > CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), > CLK(NULL, "aess_fck", &aess_fck, CK_443X), > - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { > CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > +static struct omap_clk omap443x_clks[] = { > + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > + CLK(NULL, NULL, NULL, CK_443X), > +}; > + > + > +static struct omap_clk omap446x_clks[] = { > + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > + CLK(NULL, NULL, NULL, CK_446X), > }; > > int __init omap4xxx_clk_init(void) > { > - struct omap_clk *c; > - u32 cpu_clkflg; > - > if (cpu_is_omap443x()) { > cpu_mask = RATE_IN_4430; > - cpu_clkflg = CK_443X; > } else if (cpu_is_omap446x() || cpu_is_omap447x()) { > cpu_mask = RATE_IN_4460 | RATE_IN_4430; > - cpu_clkflg = CK_446X | CK_443X; > > if (cpu_is_omap447x()) > pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); > @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) > * omap2_clk_disable_clkdm_control(); > */ > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - clk_preinit(c->lk.clk); > - > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > - c++) > - if (c->cpu & cpu_clkflg) { > - clkdev_add(&c->lk); > - clk_register(c->lk.clk); > - omap2_init_clk_clkdm(c->lk.clk); > - } > + omap2_clks_register(omap44xx_clks); > + > + if (cpu_is_omap443x()) > + omap2_clks_register(omap443x_clks); > + else if (cpu_is_omap446x() || cpu_is_omap447x()) > + omap2_clks_register(omap446x_clks); > + else > + return 0; > > /* Disable autoidle on all clocks; let the PM code enable it later */ > omap_clk_disable_autoidle_all(); > -- > 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: RE: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API 2012-10-19 10:34 ` Strashko, Grygorii @ 2012-10-22 21:17 ` Mike Turquette -1 siblings, 0 replies; 12+ messages in thread From: Mike Turquette @ 2012-10-22 21:17 UTC (permalink / raw) To: Strashko, Grygorii, Paul Walmsley Cc: Hilman, Kevin, Tony Lindgren, linux-omap, linux-arm-kernel Quoting Strashko, Grygorii (2012-10-19 03:34:19) > Hi Mike, > > > In addition to removing CK_443X/CK_446X you could also get rid of struct > > omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and > > the CLK macro by changing the clk array to be of type struct clk_lookup > > and using the CLKDEV_INIT macro. > Yes. It may be done. As I understand, this approach is applicable for you, if so i can continue working on it. right? > > > I wonder if splitting the tables would make initdata any easier. It > OMAP5,4,2 - there are no so much differences between SOcs, so it looks good and simple. > OMAP3 - it is a little bit more complex. > > would be nice to get more functional changes out of this series > > alongside the data reorganization. > Could you provide more details, pls? > What I mean to say is that this patch doesn't change any functionality. It just reorganizes data to make things prettier. In order to justify getting this change upstream (and to combat accusations of "needless churn") it would be good if this series did more than just shuffle the data around. Two examples I have already mentioned are getting rid of the macros in ckdev_omap.h (and maybe getting rid of that entire file) and also marking data as initdata. Both are functional changes that make the world a better place. Regards, Mike > > Also have you looked at splitting the tables for OMAP2/3? > Yes. I think, it can be done. > > Best regards, > Grygorii Strashko | GlobalLogic > > ________________________________________ > From: Mike Turquette [mturquette@linaro.org] > Sent: Thursday, October 18, 2012 8:53 PM > To: Strashko, Grygorii; Paul Walmsley > Cc: Hilman, Kevin; Tony Lindgren; linux-omap@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Strashko, Grygorii > Subject: Re: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API > > Quoting Grygorii Strashko (2012-10-18 09:38:17) > > Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks > > array and add corresponding set of clocks per each SoC: > > - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 > > - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x > > - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x > > and don't rely on platform specific flags any more. > > > > Use omap4_clks_register() API to register and init OMAP4 set of > > clocks. > > > > With this change, we can now safely remove CK_443X/CK_446X etc macro > > usage. It has not been done in this patch, but if the approach is OK, > > then, it is possible to do the same. > > > > In addition to removing CK_443X/CK_446X you could also get rid of struct > omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and > the CLK macro by changing the clk array to be of type struct clk_lookup > and using the CLKDEV_INIT macro. > > > One additional benefit seen is that the match logic can entirely be > > skipped. > > > > I wonder if splitting the tables would make initdata any easier. It > would be nice to get more functional changes out of this series > alongside the data reorganization. > > Also have you looked at splitting the tables for OMAP2/3? > > Regards, > Mike > > > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > > --- > > arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- > > 1 file changed, 21 insertions(+), 19 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > > index 6efc30c..4060c9c 100644 > > --- a/arch/arm/mach-omap2/clock44xx_data.c > > +++ b/arch/arm/mach-omap2/clock44xx_data.c > > @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { > > CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), > > CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), > > CLK(NULL, "aess_fck", &aess_fck, CK_443X), > > - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > > - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > > @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { > > CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > > CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > > CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), > > + CLK(NULL, NULL, NULL, CK_443X), > > +}; > > + > > +static struct omap_clk omap443x_clks[] = { > > + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > > + CLK(NULL, NULL, NULL, CK_443X), > > +}; > > + > > + > > +static struct omap_clk omap446x_clks[] = { > > + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > > + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > > + CLK(NULL, NULL, NULL, CK_446X), > > }; > > > > int __init omap4xxx_clk_init(void) > > { > > - struct omap_clk *c; > > - u32 cpu_clkflg; > > - > > if (cpu_is_omap443x()) { > > cpu_mask = RATE_IN_4430; > > - cpu_clkflg = CK_443X; > > } else if (cpu_is_omap446x() || cpu_is_omap447x()) { > > cpu_mask = RATE_IN_4460 | RATE_IN_4430; > > - cpu_clkflg = CK_446X | CK_443X; > > > > if (cpu_is_omap447x()) > > pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); > > @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) > > * omap2_clk_disable_clkdm_control(); > > */ > > > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > > - c++) > > - clk_preinit(c->lk.clk); > > - > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > > - c++) > > - if (c->cpu & cpu_clkflg) { > > - clkdev_add(&c->lk); > > - clk_register(c->lk.clk); > > - omap2_init_clk_clkdm(c->lk.clk); > > - } > > + omap2_clks_register(omap44xx_clks); > > + > > + if (cpu_is_omap443x()) > > + omap2_clks_register(omap443x_clks); > > + else if (cpu_is_omap446x() || cpu_is_omap447x()) > > + omap2_clks_register(omap446x_clks); > > + else > > + return 0; > > > > /* Disable autoidle on all clocks; let the PM code enable it later */ > > omap_clk_disable_autoidle_all(); > > -- > > 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API @ 2012-10-22 21:17 ` Mike Turquette 0 siblings, 0 replies; 12+ messages in thread From: Mike Turquette @ 2012-10-22 21:17 UTC (permalink / raw) To: linux-arm-kernel Quoting Strashko, Grygorii (2012-10-19 03:34:19) > Hi Mike, > > > In addition to removing CK_443X/CK_446X you could also get rid of struct > > omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and > > the CLK macro by changing the clk array to be of type struct clk_lookup > > and using the CLKDEV_INIT macro. > Yes. It may be done. As I understand, this approach is applicable for you, if so i can continue working on it. right? > > > I wonder if splitting the tables would make initdata any easier. It > OMAP5,4,2 - there are no so much differences between SOcs, so it looks good and simple. > OMAP3 - it is a little bit more complex. > > would be nice to get more functional changes out of this series > > alongside the data reorganization. > Could you provide more details, pls? > What I mean to say is that this patch doesn't change any functionality. It just reorganizes data to make things prettier. In order to justify getting this change upstream (and to combat accusations of "needless churn") it would be good if this series did more than just shuffle the data around. Two examples I have already mentioned are getting rid of the macros in ckdev_omap.h (and maybe getting rid of that entire file) and also marking data as initdata. Both are functional changes that make the world a better place. Regards, Mike > > Also have you looked at splitting the tables for OMAP2/3? > Yes. I think, it can be done. > > Best regards, > Grygorii Strashko | GlobalLogic > > ________________________________________ > From: Mike Turquette [mturquette at linaro.org] > Sent: Thursday, October 18, 2012 8:53 PM > To: Strashko, Grygorii; Paul Walmsley > Cc: Hilman, Kevin; Tony Lindgren; linux-omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org; Strashko, Grygorii > Subject: Re: [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API > > Quoting Grygorii Strashko (2012-10-18 09:38:17) > > Remove OMAP443x and OMAP446x specific clocks from omap44xx_clks > > array and add corresponding set of clocks per each SoC: > > - struct omap_clk omap44xx_clks[]; - common clocks set for all OMAP4 > > - struct omap_clk omap443x_clks[]; - specific clocks set for OMAP443x > > - struct omap_clk omap446x_clks[]; - specific clocks set for OMAP446x > > and don't rely on platform specific flags any more. > > > > Use omap4_clks_register() API to register and init OMAP4 set of > > clocks. > > > > With this change, we can now safely remove CK_443X/CK_446X etc macro > > usage. It has not been done in this patch, but if the approach is OK, > > then, it is possible to do the same. > > > > In addition to removing CK_443X/CK_446X you could also get rid of struct > omap_clk completely (arch/arm/plat-omap/include/plat/clkdev_omap.h) and > the CLK macro by changing the clk array to be of type struct clk_lookup > and using the CLKDEV_INIT macro. > > > One additional benefit seen is that the match logic can entirely be > > skipped. > > > > I wonder if splitting the tables would make initdata any easier. It > would be nice to get more functional changes out of this series > alongside the data reorganization. > > Also have you looked at splitting the tables for OMAP2/3? > > Regards, > Mike > > > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > > --- > > arch/arm/mach-omap2/clock44xx_data.c | 40 ++++++++++++++++++---------------- > > 1 file changed, 21 insertions(+), 19 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > > index 6efc30c..4060c9c 100644 > > --- a/arch/arm/mach-omap2/clock44xx_data.c > > +++ b/arch/arm/mach-omap2/clock44xx_data.c > > @@ -3145,10 +3145,7 @@ static struct omap_clk omap44xx_clks[] = { > > CLK(NULL, "aes1_fck", &aes1_fck, CK_443X), > > CLK(NULL, "aes2_fck", &aes2_fck, CK_443X), > > CLK(NULL, "aess_fck", &aess_fck, CK_443X), > > - CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > > - CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > > @@ -3346,19 +3343,27 @@ static struct omap_clk omap44xx_clks[] = { > > CLK("4903c000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > > CLK("4903e000.timer", "timer_sys_ck", &syc_clk_div_ck, CK_443X), > > CLK(NULL, "cpufreq_ck", &dpll_mpu_ck, CK_443X), > > + CLK(NULL, NULL, NULL, CK_443X), > > +}; > > + > > +static struct omap_clk omap443x_clks[] = { > > + CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > > + CLK(NULL, NULL, NULL, CK_443X), > > +}; > > + > > + > > +static struct omap_clk omap446x_clks[] = { > > + CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > > + CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > > + CLK(NULL, NULL, NULL, CK_446X), > > }; > > > > int __init omap4xxx_clk_init(void) > > { > > - struct omap_clk *c; > > - u32 cpu_clkflg; > > - > > if (cpu_is_omap443x()) { > > cpu_mask = RATE_IN_4430; > > - cpu_clkflg = CK_443X; > > } else if (cpu_is_omap446x() || cpu_is_omap447x()) { > > cpu_mask = RATE_IN_4460 | RATE_IN_4430; > > - cpu_clkflg = CK_446X | CK_443X; > > > > if (cpu_is_omap447x()) > > pr_warn("WARNING: OMAP4470 clock data incomplete!\n"); > > @@ -3375,17 +3380,14 @@ int __init omap4xxx_clk_init(void) > > * omap2_clk_disable_clkdm_control(); > > */ > > > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > > - c++) > > - clk_preinit(c->lk.clk); > > - > > - for (c = omap44xx_clks; c < omap44xx_clks + ARRAY_SIZE(omap44xx_clks); > > - c++) > > - if (c->cpu & cpu_clkflg) { > > - clkdev_add(&c->lk); > > - clk_register(c->lk.clk); > > - omap2_init_clk_clkdm(c->lk.clk); > > - } > > + omap2_clks_register(omap44xx_clks); > > + > > + if (cpu_is_omap443x()) > > + omap2_clks_register(omap443x_clks); > > + else if (cpu_is_omap446x() || cpu_is_omap447x()) > > + omap2_clks_register(omap446x_clks); > > + else > > + return 0; > > > > /* Disable autoidle on all clocks; let the PM code enable it later */ > > omap_clk_disable_autoidle_all(); > > -- > > 1.7.9.5 ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2012-10-22 21:17 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-10-18 16:38 [RFC PATCH 0/2] ARM: OMAP4+: Clk registration update Grygorii Strashko 2012-10-18 16:38 ` Grygorii Strashko 2012-10-18 16:38 ` [RFC PATCH 1/2] ARM: OMAP2+: clock: Add omap2_clks_register API Grygorii Strashko 2012-10-18 16:38 ` Grygorii Strashko 2012-10-18 16:38 ` [RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API Grygorii Strashko 2012-10-18 16:38 ` Grygorii Strashko 2012-10-18 17:53 ` Mike Turquette 2012-10-18 17:53 ` Mike Turquette 2012-10-19 10:34 ` Strashko, Grygorii 2012-10-19 10:34 ` Strashko, Grygorii 2012-10-22 21:17 ` Mike Turquette 2012-10-22 21:17 ` Mike Turquette
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.