diff mbox

cpufreq: Reformat printk() statement

Message ID 1394127279-29871-1-git-send-email-soren.brinkmann@xilinx.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Soren Brinkmann March 6, 2014, 5:34 p.m. UTC
Reformat a printk statement to:
 - use pr_warn
 - bring the whole string into a single line in favor of being able to
   grep for the message (ignoring the 80 char limit)

Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
---
 drivers/cpufreq/cpufreq.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

Comments

Viresh Kumar March 10, 2014, 3:53 a.m. UTC | #1
On 7 March 2014 01:34, Soren Brinkmann <soren.brinkmann@xilinx.com> wrote:
> Reformat a printk statement to:
>  - use pr_warn
>  - bring the whole string into a single line in favor of being able to
>    grep for the message (ignoring the 80 char limit)
>
> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
> ---
>  drivers/cpufreq/cpufreq.c | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> index cb003a6b72c8..534c2df608ed 100644
> --- a/drivers/cpufreq/cpufreq.c
> +++ b/drivers/cpufreq/cpufreq.c
> @@ -1874,11 +1874,8 @@ static int __cpufreq_governor(struct cpufreq_policy *policy,
>                 if (!gov)
>                         return -EINVAL;
>                 else {
> -                       printk(KERN_WARNING "%s governor failed, too long"
> -                              " transition latency of HW, fallback"
> -                              " to %s governor\n",
> -                              policy->governor->name,
> -                              gov->name);
> +                       pr_warn("%s governor failed, too long transition latency of HW, fallback to %s governor\n",
> +                                       policy->governor->name, gov->name);
>                         policy->governor = gov;
>                 }
>         }

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index cb003a6b72c8..534c2df608ed 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -1874,11 +1874,8 @@  static int __cpufreq_governor(struct cpufreq_policy *policy,
 		if (!gov)
 			return -EINVAL;
 		else {
-			printk(KERN_WARNING "%s governor failed, too long"
-			       " transition latency of HW, fallback"
-			       " to %s governor\n",
-			       policy->governor->name,
-			       gov->name);
+			pr_warn("%s governor failed, too long transition latency of HW, fallback to %s governor\n",
+					policy->governor->name, gov->name);
 			policy->governor = gov;
 		}
 	}