summaryrefslogtreecommitdiff
path: root/drivers/thermal/rockchip_thermal.c
diff options
context:
space:
mode:
authorBrian Norris <briannorris@chromium.org>2016-12-12 14:05:31 +0300
committerEduardo Valentin <edubezval@gmail.com>2016-12-14 07:31:54 +0300
commite6ed1b4ad30331e6d878579dd95764d0a224cacd (patch)
treea9579efe532f70986767047b39123a0b5104beca /drivers/thermal/rockchip_thermal.c
parentcdb98c2698b4af287925abcba4d77d92af82a0c3 (diff)
downloadlinux-e6ed1b4ad30331e6d878579dd95764d0a224cacd.tar.xz
thermal: rockchip: improve conversion error messages
These error messages don't give much information about what went wrong. It would be nice, for one, to see what invalid temperature was being requested when conversion fails. It's also good to return an error when we can't handle a conversion properly. While we're at it, fix the grammar too. Signed-off-by: Brian Norris <briannorris@chromium.org> Signed-off-by: Caesar Wang <wxt@rock-chips.com> Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
Diffstat (limited to 'drivers/thermal/rockchip_thermal.c')
-rw-r--r--drivers/thermal/rockchip_thermal.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/thermal/rockchip_thermal.c b/drivers/thermal/rockchip_thermal.c
index b811b0fb61b1..3bbc97c67051 100644
--- a/drivers/thermal/rockchip_thermal.c
+++ b/drivers/thermal/rockchip_thermal.c
@@ -424,7 +424,8 @@ static u32 rk_tsadcv2_temp_to_code(struct chip_tsadc_table table,
}
exit:
- pr_err("Invalid the conversion, error=%d\n", error);
+ pr_err("%s: invalid temperature, temp=%d error=%d\n",
+ __func__, temp, error);
return error;
}
@@ -475,7 +476,8 @@ static int rk_tsadcv2_code_to_temp(struct chip_tsadc_table table, u32 code,
}
break;
default:
- pr_err("Invalid the conversion table\n");
+ pr_err("%s: unknown table mode: %d\n", __func__, table.mode);
+ return -EINVAL;
}
/*