[PATCH 1/4] clk:spear1340:Fix: Rename clk ids within predefined limit

viresh kumar viresh.kumar at linaro.org
Fri Jul 13 04:06:58 EDT 2012


On Fri, Jul 13, 2012 at 7:23 AM, Vipul Kumar Samar
<vipulkumar.samar at st.com> wrote:
> The max limit of con_id is 16 and dev_id is 20. As of now for spear1340, many
> clk ids are exceeding this predefined limit.
>
> diff --git a/drivers/clk/spear/spear1340_clock.c b/drivers/clk/spear/spear1340_clock.c

> @@ -509,23 +507,23 @@ void __init spear1340_clk_init(void)

> -       clk_register_clkdev(clk1, "uart0_synth_gate_clk", NULL);

> -       clk_register_clkdev(clk1, "sdhci_synth_gate_clk", NULL);

> -       clk_register_clkdev(clk, "gmac_phy_input_mux_clk", NULL);
> -       clk_register_clkdev(clk1, "gmac_phy_synth_gate_clk", NULL);

These are the only places in this entire patch, where i found con_id
crossing the limit.
Then why so many changes? Aligning all to similar naming?


>         clk_register_clkdev(clk, NULL, "e0000000.serial");

@Russell & Mike: Above line uses maximum length of dev_id, i.e. 16.
And because with DT we have so long device names now, out of which
8 - address
1 - '.'
1 - '\0'

10 are already gone and so only 6 are left for device name. Should we
consider updating
dev_id array size??

--
viresh



More information about the linux-arm-kernel mailing list