Commit 603375df authored by Jishnu Prakash's avatar Jishnu Prakash Committed by Jonathan Cameron

iio: adc: Update debug prints

Change pr_err/pr_debug statements to dev_err/dev_dbg for
increased clarity.
Signed-off-by: default avatarJishnu Prakash <jprakash@codeaurora.org>
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 35fbb7f0
...@@ -246,11 +246,11 @@ static int adc5_read_voltage_data(struct adc5_chip *adc, u16 *data) ...@@ -246,11 +246,11 @@ static int adc5_read_voltage_data(struct adc5_chip *adc, u16 *data)
*data = (rslt_msb << 8) | rslt_lsb; *data = (rslt_msb << 8) | rslt_lsb;
if (*data == ADC5_USR_DATA_CHECK) { if (*data == ADC5_USR_DATA_CHECK) {
pr_err("Invalid data:0x%x\n", *data); dev_err(adc->dev, "Invalid data:0x%x\n", *data);
return -EINVAL; return -EINVAL;
} }
pr_debug("voltage raw code:0x%x\n", *data); dev_dbg(adc->dev, "voltage raw code:0x%x\n", *data);
return 0; return 0;
} }
...@@ -382,24 +382,24 @@ static int adc5_do_conversion(struct adc5_chip *adc, ...@@ -382,24 +382,24 @@ static int adc5_do_conversion(struct adc5_chip *adc,
ret = adc5_configure(adc, prop); ret = adc5_configure(adc, prop);
if (ret) { if (ret) {
pr_err("ADC configure failed with %d\n", ret); dev_err(adc->dev, "ADC configure failed with %d\n", ret);
goto unlock; goto unlock;
} }
if (adc->poll_eoc) { if (adc->poll_eoc) {
ret = adc5_poll_wait_eoc(adc); ret = adc5_poll_wait_eoc(adc);
if (ret) { if (ret) {
pr_err("EOC bit not set\n"); dev_err(adc->dev, "EOC bit not set\n");
goto unlock; goto unlock;
} }
} else { } else {
ret = wait_for_completion_timeout(&adc->complete, ret = wait_for_completion_timeout(&adc->complete,
ADC5_CONV_TIMEOUT); ADC5_CONV_TIMEOUT);
if (!ret) { if (!ret) {
pr_debug("Did not get completion timeout.\n"); dev_dbg(adc->dev, "Did not get completion timeout.\n");
ret = adc5_poll_wait_eoc(adc); ret = adc5_poll_wait_eoc(adc);
if (ret) { if (ret) {
pr_err("EOC bit not set\n"); dev_err(adc->dev, "EOC bit not set\n");
goto unlock; goto unlock;
} }
} }
...@@ -721,7 +721,7 @@ static int adc5_get_dt_channel_data(struct adc5_chip *adc, ...@@ -721,7 +721,7 @@ static int adc5_get_dt_channel_data(struct adc5_chip *adc,
channel_name = of_get_property(node, channel_name = of_get_property(node,
"label", NULL) ? : node->name; "label", NULL) ? : node->name;
if (!channel_name) { if (!channel_name) {
pr_err("Invalid channel name\n"); dev_err(dev, "Invalid channel name\n");
return -EINVAL; return -EINVAL;
} }
prop->datasheet_name = channel_name; prop->datasheet_name = channel_name;
...@@ -764,7 +764,7 @@ static int adc5_get_dt_channel_data(struct adc5_chip *adc, ...@@ -764,7 +764,7 @@ static int adc5_get_dt_channel_data(struct adc5_chip *adc,
return ret; return ret;
} }
pr_debug("dig_ver:minor:%d, major:%d\n", dig_version[0], dev_dbg(dev, "dig_ver:minor:%d, major:%d\n", dig_version[0],
dig_version[1]); dig_version[1]);
/* Digital controller >= 5.3 have hw_settle_2 option */ /* Digital controller >= 5.3 have hw_settle_2 option */
if ((dig_version[0] >= ADC5_HW_SETTLE_DIFF_MINOR && if ((dig_version[0] >= ADC5_HW_SETTLE_DIFF_MINOR &&
...@@ -966,7 +966,7 @@ static int adc5_probe(struct platform_device *pdev) ...@@ -966,7 +966,7 @@ static int adc5_probe(struct platform_device *pdev)
ret = adc5_get_dt_data(adc, node); ret = adc5_get_dt_data(adc, node);
if (ret) { if (ret) {
pr_err("adc get dt data failed\n"); dev_err(dev, "adc get dt data failed\n");
return ret; return ret;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment