Re: [PATCH] cpufreq: Reformat printk() statement

[Viresh Kumar]

On 7 March 2014 01:34, Soren Brinkmann <soren.brinkmann@xxxxxxxxxx> 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@xxxxxxxxxx>
> ---
>  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@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/