[v4,2/8] mfd: hi6421-pmic: cleanup: update dev_err messages

Message ID 20170606085143.13154-3-guodong.xu@linaro.org
State New
Headers show
Series
  • MFD: add driver for HiSilicon Hi6421v530 PMIC
Related show

Commit Message

Guodong Xu June 6, 2017, 8:51 a.m.
Update dev_err messages to make them more readable.

Signed-off-by: Guodong Xu <guodong.xu@linaro.org>

---
 drivers/mfd/hi6421-pmic-core.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

-- 
2.10.2

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Lee Jones June 6, 2017, 1:50 p.m. | #1
On Tue, 06 Jun 2017, Guodong Xu wrote:

> Update dev_err messages to make them more readable.

> 

> Signed-off-by: Guodong Xu <guodong.xu@linaro.org>

> ---

>  drivers/mfd/hi6421-pmic-core.c | 6 +++---

>  1 file changed, 3 insertions(+), 3 deletions(-)


For my own reference:
  Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org>
  
> diff --git a/drivers/mfd/hi6421-pmic-core.c b/drivers/mfd/hi6421-pmic-core.c

> index ad9e3d8..b1139d4 100644

> --- a/drivers/mfd/hi6421-pmic-core.c

> +++ b/drivers/mfd/hi6421-pmic-core.c

> @@ -52,8 +52,8 @@ static int hi6421_pmic_probe(struct platform_device *pdev)

>  	pmic->regmap = devm_regmap_init_mmio_clk(&pdev->dev, NULL, base,

>  						 &hi6421_regmap_config);

>  	if (IS_ERR(pmic->regmap)) {

> -		dev_err(&pdev->dev,

> -			"regmap init failed: %ld\n", PTR_ERR(pmic->regmap));

> +		dev_err(&pdev->dev, "Failed to initialise Regmap: %ld\n",

> +						PTR_ERR(pmic->regmap));

>  		return PTR_ERR(pmic->regmap);

>  	}

>  

> @@ -70,7 +70,7 @@ static int hi6421_pmic_probe(struct platform_device *pdev)

>  	ret = devm_mfd_add_devices(&pdev->dev, 0, hi6421_devs,

>  				   ARRAY_SIZE(hi6421_devs), NULL, 0, NULL);

>  	if (ret) {

> -		dev_err(&pdev->dev, "add mfd devices failed: %d\n", ret);

> +		dev_err(&pdev->dev, "Failed to add child devices: %d\n", ret);

>  		return ret;

>  	}

>  


-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch hide | download patch | download mbox

diff --git a/drivers/mfd/hi6421-pmic-core.c b/drivers/mfd/hi6421-pmic-core.c
index ad9e3d8..b1139d4 100644
--- a/drivers/mfd/hi6421-pmic-core.c
+++ b/drivers/mfd/hi6421-pmic-core.c
@@ -52,8 +52,8 @@  static int hi6421_pmic_probe(struct platform_device *pdev)
 	pmic->regmap = devm_regmap_init_mmio_clk(&pdev->dev, NULL, base,
 						 &hi6421_regmap_config);
 	if (IS_ERR(pmic->regmap)) {
-		dev_err(&pdev->dev,
-			"regmap init failed: %ld\n", PTR_ERR(pmic->regmap));
+		dev_err(&pdev->dev, "Failed to initialise Regmap: %ld\n",
+						PTR_ERR(pmic->regmap));
 		return PTR_ERR(pmic->regmap);
 	}
 
@@ -70,7 +70,7 @@  static int hi6421_pmic_probe(struct platform_device *pdev)
 	ret = devm_mfd_add_devices(&pdev->dev, 0, hi6421_devs,
 				   ARRAY_SIZE(hi6421_devs), NULL, 0, NULL);
 	if (ret) {
-		dev_err(&pdev->dev, "add mfd devices failed: %d\n", ret);
+		dev_err(&pdev->dev, "Failed to add child devices: %d\n", ret);
 		return ret;
 	}