From: Markus Elfring <[email protected]> Date: Wed, 27 Sep 2017 18:40:34 +0200
Omit extra messages for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <[email protected]> --- drivers/clk/tegra/clk-audio-sync.c | 4 +--- drivers/clk/tegra/clk-divider.c | 5 +---- drivers/clk/tegra/clk-periph-gate.c | 4 +--- 3 files changed, 3 insertions(+), 10 deletions(-) diff --git a/drivers/clk/tegra/clk-audio-sync.c b/drivers/clk/tegra/clk-audio-sync.c index 92d04ce2dee6..62a84c883dd4 100644 --- a/drivers/clk/tegra/clk-audio-sync.c +++ b/drivers/clk/tegra/clk-audio-sync.c @@ -62,10 +62,8 @@ struct clk *tegra_clk_register_sync_source(const char *name, struct clk *clk; sync = kzalloc(sizeof(*sync), GFP_KERNEL); - if (!sync) { - pr_err("%s: could not allocate sync source clk\n", __func__); + if (!sync) return ERR_PTR(-ENOMEM); - } sync->rate = rate; sync->max_rate = max_rate; diff --git a/drivers/clk/tegra/clk-divider.c b/drivers/clk/tegra/clk-divider.c index 16e0aee14773..5ee9423ccac2 100644 --- a/drivers/clk/tegra/clk-divider.c +++ b/drivers/clk/tegra/clk-divider.c @@ -156,11 +156,8 @@ struct clk *tegra_clk_register_divider(const char *name, struct clk_init_data init; divider = kzalloc(sizeof(*divider), GFP_KERNEL); - if (!divider) { - pr_err("%s: could not allocate fractional divider clk\n", - __func__); + if (!divider) return ERR_PTR(-ENOMEM); - } init.name = name; init.ops = &tegra_clk_frac_div_ops; diff --git a/drivers/clk/tegra/clk-periph-gate.c b/drivers/clk/tegra/clk-periph-gate.c index 303ef32ee3f1..276cc4c59204 100644 --- a/drivers/clk/tegra/clk-periph-gate.c +++ b/drivers/clk/tegra/clk-periph-gate.c @@ -141,10 +141,8 @@ struct clk *tegra_clk_register_periph_gate(const char *name, return ERR_PTR(-EINVAL); gate = kzalloc(sizeof(*gate), GFP_KERNEL); - if (!gate) { - pr_err("%s: could not allocate periph gate clk\n", __func__); + if (!gate) return ERR_PTR(-ENOMEM); - } init.name = name; init.flags = flags; -- 2.14.2
