sh: clkfwk: use '"%s:", __func__' instead of function name.

Commit Message

Arvind Yadav Oct. 9, 2017, 12:15 p.m. UTC

The script "checkpatch.pl" pointed information out like the following.
WARNING: Prefer using '"%s...", __func__' to using 'sh_clk_init_parent',
this function's name, in a string.

pr_err() strings should end with newlines. Add '\n' to fix this.

Signed-off-by: Arvind Yadav <[e-mail]>
---
 drivers/sh/clk/cpg.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Patch

diff --git a/drivers/sh/clk/cpg.c b/drivers/sh/clk/cpg.c
index 7442bc1..4b5b2f6 100644
--- a/drivers/sh/clk/cpg.c
+++ b/drivers/sh/clk/cpg.c
@@ -217,7 +217,7 @@  static int __init sh_clk_init_parent(struct clk *clk)
 		return 0;
 
 	if (!clk->src_width) {
-		pr_err("sh_clk_init_parent: cannot select parent clock\n");
+		pr_err("%s: cannot select parent clock\n", __func__);
 		return -EINVAL;
 	}
 
@@ -225,13 +225,13 @@  static int __init sh_clk_init_parent(struct clk *clk)
 	val &= (1 << clk->src_width) - 1;
 
 	if (val >= clk->parent_num) {
-		pr_err("sh_clk_init_parent: parent table size failed\n");
+		pr_err("%s: parent table size failed\n", __func__);
 		return -EINVAL;
 	}
 
 	clk_reparent(clk, clk->parent_table[val]);
 	if (!clk->parent) {
-		pr_err("sh_clk_init_parent: unable to set parent");
+		pr_err("%s: unable to set parent\n", __func__);
 		return -EINVAL;
 	}