[PATCH 1/2] drivers: regulator: core: use pr_fmt

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Daniel Walker
Date: Wednesday, November 17, 2010 - 4:30 pm

This adds a pr_fmt line which uses the __func__ macro. I also
convert the current pr_ lines to remove their __func__ usage.

Cc: bleong@codeaurora.org
Signed-off-by: Daniel Walker <dwalker@codeaurora.org>
---
 drivers/regulator/core.c |   22 ++++++++++++----------
 1 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
index f1d10c9..a3ca3d6 100644
--- a/drivers/regulator/core.c
+++ b/drivers/regulator/core.c
@@ -13,6 +13,8 @@
  *
  */
 
+#define pr_fmt(fmt) "%s:" fmt, __func__
+
 #include <linux/kernel.h>
 #include <linux/init.h>
 #include <linux/device.h>
@@ -765,8 +767,8 @@ static int machine_constraints_voltage(struct regulator_dev *rdev,
 
 		/* else require explicit machine-level constraints */
 		if (cmin <= 0 || cmax <= 0 || cmax < cmin) {
-			pr_err("%s: %s '%s' voltage constraints\n",
-				       __func__, "invalid", name);
+			pr_err("%s '%s' voltage constraints\n", "invalid",
+				name);
 			return -EINVAL;
 		}
 
@@ -787,22 +789,22 @@ static int machine_constraints_voltage(struct regulator_dev *rdev,
 
 		/* final: [min_uV..max_uV] valid iff constraints valid */
 		if (max_uV < min_uV) {
-			pr_err("%s: %s '%s' voltage constraints\n",
-				       __func__, "unsupportable", name);
+			pr_err("%s '%s' voltage constraints\n", "unsupportable",
+				name);
 			return -EINVAL;
 		}
 
 		/* use regulator's subset of machine constraints */
 		if (constraints->min_uV < min_uV) {
-			pr_debug("%s: override '%s' %s, %d -> %d\n",
-				       __func__, name, "min_uV",
-					constraints->min_uV, min_uV);
+			pr_debug("override '%s' %s, %d -> %d\n",
+				 name, "min_uV",
+				 constraints->min_uV, min_uV);
 			constraints->min_uV = min_uV;
 		}
 		if (constraints->max_uV > max_uV) {
-			pr_debug("%s: override '%s' %s, %d -> %d\n",
-				       __func__, name, "max_uV",
-					constraints->max_uV, max_uV);
+			pr_debug("override '%s' %s, %d -> %d\n",
+				 name, "max_uV",
+				 constraints->max_uV, max_uV);
 			constraints->max_uV = max_uV;
 		}
 	}
-- 
1.7.1

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 1/2] drivers: regulator: core: use pr_fmt, Daniel Walker, (Wed Nov 17, 4:30 pm)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Joe Perches, (Wed Nov 17, 5:29 pm)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Daniel Walker, (Wed Nov 17, 5:35 pm)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Mark Brown, (Thu Nov 18, 3:35 am)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Mark Brown, (Thu Nov 18, 3:37 am)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Joe Perches, (Thu Nov 18, 9:44 am)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Liam Girdwood, (Fri Nov 19, 4:09 am)
Re: [PATCH 1/2] drivers: regulator: core: use pr_fmt, Joe Perches, (Fri Nov 19, 10:07 am)