clk: h8300: Drop allocation printk and cleanup sizeof style
authorStephen Boyd <sboyd@codeaurora.org>
Mon, 13 Jul 2015 23:54:04 +0000 (16:54 -0700)
committerStephen Boyd <sboyd@codeaurora.org>
Tue, 28 Jul 2015 18:51:36 +0000 (11:51 -0700)
We don't need to print an error on allocation failures, drop it.
While we're here, change the sizeof() to be sizeof(*<ptr>) to
make code more future proof.

Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
drivers/clk/h8300/clk-h8s2678.c

index 4701b09..c121eb5 100644 (file)
@@ -98,11 +98,9 @@ static void __init h8s2678_pll_clk_setup(struct device_node *node)
        }
 
 
-       pll_clock = kzalloc(sizeof(struct pll_clock), GFP_KERNEL);
-       if (!pll_clock) {
-               pr_err("%s: failed to alloc memory", clk_name);
+       pll_clock = kzalloc(sizeof(*pll_clock), GFP_KERNEL);
+       if (!pll_clock)
                return;
-       }
 
        pll_clock->sckcr = of_iomap(node, 0);
        if (pll_clock->sckcr == NULL) {