All of lore.kernel.org
 help / color / mirror / Atom feed
* Patch "iio: adc: cc10001: Fix regulator_get_voltage() return value check" has been added to the 4.0-stable tree
@ 2015-06-03  1:15 gregkh
  0 siblings, 0 replies; only message in thread
From: gregkh @ 2015-06-03  1:15 UTC (permalink / raw)
  To: naidu.tellapati, ezequiel.garcia, gregkh, jic23; +Cc: stable, stable-commits


This is a note to let you know that I've just added the patch titled

    iio: adc: cc10001: Fix regulator_get_voltage() return value check

to the 4.0-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     iio-adc-cc10001-fix-regulator_get_voltage-return-value-check.patch
and it can be found in the queue-4.0 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@vger.kernel.org> know about it.


>From 65a761bf8d55fdcf8ecc4642382a4e76c086e44c Mon Sep 17 00:00:00 2001
From: Naidu Tellapati <naidu.tellapati@imgtec.com>
Date: Thu, 7 May 2015 18:22:19 -0300
Subject: iio: adc: cc10001: Fix regulator_get_voltage() return value check

From: Naidu Tellapati <naidu.tellapati@imgtec.com>

commit 65a761bf8d55fdcf8ecc4642382a4e76c086e44c upstream.

regulator_get_voltage() returns a non-negative value in case of success,
and a negative error in case of error. Let's fix this.

Fixes: 1664f6a5b0c8 ("iio: adc: Cosmic Circuits 10001 ADC driver")
Signed-off-by: Naidu Tellapati <naidu.tellapati@imgtec.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@imgtec.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

---
 drivers/iio/adc/cc10001_adc.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/iio/adc/cc10001_adc.c
+++ b/drivers/iio/adc/cc10001_adc.c
@@ -231,7 +231,7 @@ static int cc10001_adc_read_raw(struct i
 
 	case IIO_CHAN_INFO_SCALE:
 		ret = regulator_get_voltage(adc_dev->reg);
-		if (ret)
+		if (ret < 0)
 			return ret;
 
 		*val = ret / 1000;


Patches currently in stable-queue which might be from naidu.tellapati@imgtec.com are

queue-4.0/iio-adc-cc10001-fix-the-channel-number-mapping.patch
queue-4.0/iio-adc-cc10001-fix-regulator_get_voltage-return-value-check.patch
queue-4.0/iio-adc-cc10001-add-delay-before-setting-start-bit.patch
queue-4.0/iio-adc-cc10001-fix-incorrect-use-of-power-up-power-down-register.patch

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2015-06-03  1:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-03  1:15 Patch "iio: adc: cc10001: Fix regulator_get_voltage() return value check" has been added to the 4.0-stable tree gregkh

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.