[RFC PATCH 2/2] ARM: OMAP4: clock: use omap4_clks_register API
Strashko, Grygorii
grygorii.strashko at ti.com
Fri Oct 19 06:34:19 EDT 2012
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 at 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
More information about the linux-arm-kernel
mailing list