From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f67.google.com ([74.125.82.67]:36087 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754885AbcG0J3N (ORCPT ); Wed, 27 Jul 2016 05:29:13 -0400 MIME-Version: 1.0 In-Reply-To: <1469484416-17305-1-git-send-email-colin.king@canonical.com> References: <1469484416-17305-1-git-send-email-colin.king@canonical.com> From: Matt Ranostay Date: Wed, 27 Jul 2016 02:28:26 -0700 Message-ID: Subject: Re: [PATCH] iio: chemical: atlas-ph-sensor: fix typo in val assignment To: Colin King Cc: Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Sanchayan Maity , Gregor Boirie , "linux-iio@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On Mon, Jul 25, 2016 at 3:06 PM, Colin King wrote: > From: Colin Ian King > > Fix an incorrect assignment due to a typo on a variable name. The > variable val2 should be assigned 100000 and not val. Yikes! Good catch.. Reviewed-By: Matt Ranostay > > Signed-off-by: Colin Ian King > --- > drivers/iio/chemical/atlas-ph-sensor.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/chemical/atlas-ph-sensor.c b/drivers/iio/chemical/atlas-ph-sensor.c > index ae038a5..407f141 100644 > --- a/drivers/iio/chemical/atlas-ph-sensor.c > +++ b/drivers/iio/chemical/atlas-ph-sensor.c > @@ -434,7 +434,7 @@ static int atlas_read_raw(struct iio_dev *indio_dev, > break; > case IIO_ELECTRICALCONDUCTIVITY: > *val = 1; /* 0.00001 */ > - *val = 100000; > + *val2 = 100000; > break; > case IIO_CONCENTRATION: > *val = 0; /* 0.000000001 */ > -- > 2.8.1 >