clk: shmobile: Fix MSTP clock array initialization
authorValentine Barshak <valentine.barshak@cogentembedded.com>
Sat, 28 Dec 2013 12:09:09 +0000 (16:09 +0400)
committerMike Turquette <mturquette@linaro.org>
Tue, 14 Jan 2014 19:35:57 +0000 (11:35 -0800)
The clks member of the clk_onecell_data structure should
point to a valid clk array (no NULL entries allowed),
and the clk_num should be equal to the number
of elements in the clks array.

The MSTP driver fails to satisfy the above conditions.
The clks array may contain NULL entries if not all
clock-indices are initialized in the device tree.
Thus, if the clock indices are interleaved we end up
with NULL pointers in-between.

The other problem is the driver uses maximum clock index
as the number of clocks, which is incorrect (less than
the actual number of clocks by 1).

Fix the first issue by pre-setting the whole clks array
with ERR_PTR(-ENOENT) pointers instead of zeros; and
use maximum clkidx + 1 as the number of clocks to fix
the other one.

This should make of_clk_src_onecell_get() return the following:
* valid clk pointers for all clocks registered;
* ERR_PTR(-EINVAL) if (idx >= clk_data->clk_num);
* ERR_PTR(-ENOENT) if the clock at the selected index was not
  initialized in the device tree (and was not registered).

Changes in V2:
* removed brackets from the one-line for loop

Signed-off-by: Valentine Barshak <valentine.barshak@cogentembedded.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Tested-by: Ben Dooks <ben.dooks@codethink.co.uk>
Signed-off-by: Mike Turquette <mturquette@linaro.org>
drivers/clk/shmobile/clk-mstp.c

index be7d0174cbfcff631ccd64f4b26616004ff33bb6..42d5912b1d25aa908314f62c4138052e9fd51a15 100644 (file)
@@ -160,7 +160,7 @@ static void __init cpg_mstp_clocks_init(struct device_node *np)
        unsigned int i;
 
        group = kzalloc(sizeof(*group), GFP_KERNEL);
-       clks = kzalloc(MSTP_MAX_CLOCKS * sizeof(*clks), GFP_KERNEL);
+       clks = kmalloc(MSTP_MAX_CLOCKS * sizeof(*clks), GFP_KERNEL);
        if (group == NULL || clks == NULL) {
                kfree(group);
                kfree(clks);
@@ -181,6 +181,9 @@ static void __init cpg_mstp_clocks_init(struct device_node *np)
                return;
        }
 
+       for (i = 0; i < MSTP_MAX_CLOCKS; ++i)
+               clks[i] = ERR_PTR(-ENOENT);
+
        for (i = 0; i < MSTP_MAX_CLOCKS; ++i) {
                const char *parent_name;
                const char *name;
@@ -208,7 +211,8 @@ static void __init cpg_mstp_clocks_init(struct device_node *np)
                clks[clkidx] = cpg_mstp_clock_register(name, parent_name,
                                                       clkidx, group);
                if (!IS_ERR(clks[clkidx])) {
-                       group->data.clk_num = max(group->data.clk_num, clkidx);
+                       group->data.clk_num = max(group->data.clk_num,
+                                                 clkidx + 1);
                        /*
                         * Register a clkdev to let board code retrieve the
                         * clock by name and register aliases for non-DT