diff mbox

PM / AVS: SmartReflex: add in missing white space in error message text

Message ID 20160912111455.29881-1-colin.king@canonical.com (mailing list archive)
State Changes Requested, archived
Headers show

Commit Message

Colin King Sept. 12, 2016, 11:14 a.m. UTC 
From: Colin Ian King <colin.king@canonical.com>

Several line wrapped literal strings are missing white spaces, add the
whitespace to fix this.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/power/avs/smartreflex.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

Comments

Kevin Hilman Oct. 18, 2016, 9:05 p.m. UTC | #1 
Colin King <colin.king@canonical.com> writes:

> From: Colin Ian King <colin.king@canonical.com>
>
> Several line wrapped literal strings are missing white spaces, add the
> whitespace to fix this.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.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;
>  	}
--
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/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;
 	}