linux-kernel - Re: [PATCH] PM / AVS: SmartReflex: add in missing white space in error message text

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <7hk2d59xl6.fsf@baylibre.com>
Date:   Tue, 18 Oct 2016 14:05:57 -0700
From:   Kevin Hilman <khilman@...libre.com>
To:     Colin King <colin.king@...onical.com>
Cc:     Nishanth Menon <nm@...com>, linux-pm@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] PM / AVS: SmartReflex: add in missing white space in error message text

Colin King <colin.king@...onical.com> writes:

> From: Colin Ian King <colin.king@...onical.com>
>
> Several line wrapped literal strings are missing white spaces, add the
> whitespace to fix this.
>
> Signed-off-by: Colin Ian King <colin.king@...onical.com>
> ---
>  drivers/power/avs/smartreflex.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)

This patch duplicates work that was already done in commit
10He2cdeaa1232d (PM / AVS: SmartReflex: Neaten logging)

Please rebase/resubmit if there are cleanups still missing.

Kevin

> diff --git a/drivers/power/avs/smartreflex.c b/drivers/power/avs/smartreflex.c
> index db9973b..40f46a8 100644
> --- a/drivers/power/avs/smartreflex.c
> +++ b/drivers/power/avs/smartreflex.c
> @@ -225,8 +224,7 @@ static int sr_late_init(struct omap_sr *sr_info)
>  
>  error:
>  	list_del(&sr_info->node);
> -	dev_err(&sr_info->pdev->dev, "%s: ERROR in registering"
> -		"interrupt handler. Smartreflex will"
> +	dev_err(&sr_info->pdev->dev, "%s: ERROR in registering interrupt handler. Smartreflex will "
>  		"not function as desired\n", __func__);
>  
>  	return ret;
> @@ -387,7 +386,7 @@ int sr_configure_errgen(struct omap_sr *sr)
>  		vpboundint_st = ERRCONFIG_VPBOUNDINTST_V2;
>  		break;
>  	default:
> -		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex"
> +		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex "
>  			"module without specifying the ip\n", __func__);
>  		return -EINVAL;
>  	}
> @@ -440,7 +439,7 @@ int sr_disable_errgen(struct omap_sr *sr)
>  		vpboundint_st = ERRCONFIG_VPBOUNDINTST_V2;
>  		break;
>  	default:
> -		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex"
> +		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex "
>  			"module without specifying the ip\n", __func__);
>  		return -EINVAL;
>  	}
> @@ -504,7 +503,7 @@ int sr_configure_minmax(struct omap_sr *sr)
>  		senp_shift = SRCONFIG_SENPENABLE_V2_SHIFT;
>  		break;
>  	default:
> -		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex"
> +		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex "
>  			"module without specifying the ip\n", __func__);
>  		return -EINVAL;
>  	}
> @@ -537,7 +536,7 @@ int sr_configure_minmax(struct omap_sr *sr)
>  			IRQENABLE_MCUBOUNDSINT | IRQENABLE_MCUDISABLEACKINT);
>  		break;
>  	default:
> -		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex"
> +		dev_err(&sr->pdev->dev, "%s: Trying to Configure smartreflex "
>  			"module without specifying the ip\n", __func__);
>  		return -EINVAL;
>  	}
> @@ -571,7 +570,7 @@ int sr_enable(struct omap_sr *sr, unsigned long volt)
>  	volt_data = omap_voltage_get_voltdata(sr->voltdm, volt);
>  
>  	if (IS_ERR(volt_data)) {
> -		dev_warn(&sr->pdev->dev, "%s: Unable to get voltage table"
> +		dev_warn(&sr->pdev->dev, "%s: Unable to get voltage table "
>  			"for nominal voltage %ld\n", __func__, volt);
>  		return PTR_ERR(volt_data);
>  	}
> @@ -704,7 +703,7 @@ void omap_sr_enable(struct voltagedomain *voltdm)
>  		return;
>  
>  	if (!sr_class || !(sr_class->enable) || !(sr_class->configure)) {
> -		dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not"
> +		dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not "
>  			"registered\n", __func__);
>  		return;
>  	}
> @@ -736,7 +735,7 @@ void omap_sr_disable(struct voltagedomain *voltdm)
>  		return;
>  
>  	if (!sr_class || !(sr_class->disable)) {
> -		dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not"
> +		dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not "
>  			"registered\n", __func__);
>  		return;
>  	}
> @@ -768,7 +767,7 @@ void omap_sr_disable_reset_volt(struct voltagedomain *voltdm)
>  		return;
>  
>  	if (!sr_class || !(sr_class->disable)) {
> -		dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not"
> +		dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not "
>  			"registered\n", __func__);
>  		return;
>  	}