]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
clk: qoriq: Fix checkpatch type OOM_MESSAGE
authorEmil Medve <Emilian.Medve@Freescale.com>
Wed, 21 Jan 2015 10:03:26 +0000 (04:03 -0600)
committerMichael Turquette <mturquette@linaro.org>
Wed, 28 Jan 2015 18:11:39 +0000 (10:11 -0800)
WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
+       if (!parent_names) {
+               pr_err("%s: could not allocate parent_names\n", __func__);

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
+       if (!cmux_clk) {
+               pr_err("%s: could not allocate cmux_clk\n", __func__);

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
+       if (!subclks) {
+               pr_err("%s: could not allocate subclks\n", __func__);

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
+       if (!onecell_data) {
+               pr_err("%s: could not allocate onecell_data\n", __func__);

Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com>
Signed-off-by: Michael Turquette <mturquette@linaro.org>
drivers/clk/clk-qoriq.c

index a1cd137651a30556a40e35f5bc0b91bad9387172..e25dea919695891419b1fbdab8b5eb583c2d7da1 100644 (file)
@@ -86,19 +86,16 @@ static void __init core_mux_init(struct device_node *np)
                return;
        }
        parent_names = kcalloc(count, sizeof(char *), GFP_KERNEL);
-       if (!parent_names) {
-               pr_err("%s: could not allocate parent_names\n", __func__);
+       if (!parent_names)
                return;
-       }
 
        for (i = 0; i < count; i++)
                parent_names[i] = of_clk_get_parent_name(np, i);
 
        cmux_clk = kzalloc(sizeof(*cmux_clk), GFP_KERNEL);
-       if (!cmux_clk) {
-               pr_err("%s: could not allocate cmux_clk\n", __func__);
+       if (!cmux_clk)
                goto err_name;
-       }
+
        cmux_clk->reg = of_iomap(np, 0);
        if (!cmux_clk->reg) {
                pr_err("%s: could not map register\n", __func__);
@@ -193,16 +190,12 @@ static void __init core_pll_init(struct device_node *np)
        }
 
        subclks = kcalloc(count, sizeof(struct clk *), GFP_KERNEL);
-       if (!subclks) {
-               pr_err("%s: could not allocate subclks\n", __func__);
+       if (!subclks)
                goto err_map;
-       }
 
        onecell_data = kzalloc(sizeof(*onecell_data), GFP_KERNEL);
-       if (!onecell_data) {
-               pr_err("%s: could not allocate onecell_data\n", __func__);
+       if (!onecell_data)
                goto err_clks;
-       }
 
        for (i = 0; i < count; i++) {
                rc = of_property_read_string_index(np, "clock-output-names",