From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] PM / AVS: SmartReflex: add in missing white space in error message text Date: Tue, 18 Oct 2016 14:05:57 -0700 Message-ID: <7hk2d59xl6.fsf@baylibre.com> References: <20160912111455.29881-1-colin.king@canonical.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <20160912111455.29881-1-colin.king@canonical.com> (Colin King's message of "Mon, 12 Sep 2016 12:14:55 +0100") Sender: linux-kernel-owner@vger.kernel.org To: Colin King Cc: Nishanth Menon , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-pm@vger.kernel.org Colin King writes: > From: Colin Ian King > > Several line wrapped literal strings are missing white spaces, add the > whitespace to fix this. > > Signed-off-by: Colin Ian King > --- > 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; > }