From: Jonas Aaberg <jonas.ab...@stericsson.com>

Reread the gpadc once upon failure.

Signed-off-by: Jonas Aaberg <jonas.ab...@stericsson.com>
Signed-off-by: Lee Jones <lee.jo...@linaro.org>
Reviewed-by: Mattias WALLIN <mattias.wal...@stericsson.com>
---
 drivers/mfd/ab8500-gpadc.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/mfd/ab8500-gpadc.c b/drivers/mfd/ab8500-gpadc.c
index 34c1d04..1b48ac1 100644
--- a/drivers/mfd/ab8500-gpadc.c
+++ b/drivers/mfd/ab8500-gpadc.c
@@ -259,6 +259,11 @@ int ab8500_gpadc_convert(struct ab8500_gpadc *gpadc, u8 
channel)
        int voltage;
 
        ad_value = ab8500_gpadc_read_raw(gpadc, channel);
+
+       /* On failure retry a second time */
+       if (ad_value < 0)
+               ad_value = ab8500_gpadc_read_raw(gpadc, channel);
+
        if (ad_value < 0) {
                dev_err(gpadc->dev, "GPADC raw value failed ch: %d\n", channel);
                return ad_value;
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to