regulators: max8998.c: use dev_err() instead of printk()
Fixes the following checkpatch warning: WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... Signed-off-by: Thiago Farina <tfarina@chromium.org> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
This commit is contained in:
parent
6dbe51c251
commit
9df19a5597
1 changed files with 6 additions and 3 deletions
|
@ -665,14 +665,16 @@ static int max8998_pmic_probe(struct platform_device *pdev)
|
|||
gpio_is_valid(pdata->buck1_set2)) {
|
||||
/* Check if SET1 is not equal to 0 */
|
||||
if (!pdata->buck1_set1) {
|
||||
printk(KERN_ERR "MAX8998 SET1 GPIO defined as 0 !\n");
|
||||
dev_err(&pdev->dev,
|
||||
"MAX8998 SET1 GPIO defined as 0 !\n");
|
||||
WARN_ON(!pdata->buck1_set1);
|
||||
ret = -EIO;
|
||||
goto err_out;
|
||||
}
|
||||
/* Check if SET2 is not equal to 0 */
|
||||
if (!pdata->buck1_set2) {
|
||||
printk(KERN_ERR "MAX8998 SET2 GPIO defined as 0 !\n");
|
||||
dev_err(&pdev->dev,
|
||||
"MAX8998 SET2 GPIO defined as 0 !\n");
|
||||
WARN_ON(!pdata->buck1_set2);
|
||||
ret = -EIO;
|
||||
goto err_out;
|
||||
|
@ -738,7 +740,8 @@ static int max8998_pmic_probe(struct platform_device *pdev)
|
|||
if (gpio_is_valid(pdata->buck2_set3)) {
|
||||
/* Check if SET3 is not equal to 0 */
|
||||
if (!pdata->buck2_set3) {
|
||||
printk(KERN_ERR "MAX8998 SET3 GPIO defined as 0 !\n");
|
||||
dev_err(&pdev->dev,
|
||||
"MAX8998 SET3 GPIO defined as 0 !\n");
|
||||
WARN_ON(!pdata->buck2_set3);
|
||||
ret = -EIO;
|
||||
goto err_out;
|
||||
|
|
Loading…
Reference in a new issue