From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:55989 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753634Ab2H0RZT (ORCPT ); Mon, 27 Aug 2012 13:25:19 -0400 Message-ID: <503BAD7C.5010608@kernel.org> Date: Mon, 27 Aug 2012 18:25:16 +0100 From: Jonathan Cameron MIME-Version: 1.0 To: "Kim, Milo" CC: Lars-Peter Clausen , "linux-kernel@vger.kernel.org" , "linux-iio@vger.kernel.org" Subject: Re: [PATCH 2/3] iio: use IIO_CHAN_INFO_RAW rather than 0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 08/16/2012 08:39 AM, Kim, Milo wrote: > (a) For better readability, replace 0 with IIO_CHAN_INFO_RAW. > (b) Make same line-format as other apis() > : iio_read_channel_scale() and iio_read_channel_offset() > merged. Thanks. > Signed-off-by: Milo(Woogyom) Kim > --- > drivers/iio/inkern.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/iio/inkern.c b/drivers/iio/inkern.c > index bd09245..5103337 100644 > --- a/drivers/iio/inkern.c > +++ b/drivers/iio/inkern.c > @@ -239,8 +239,10 @@ int iio_read_channel_raw(struct iio_channel *chan, int *val) > goto err_unlock; > } > > - ret = chan->indio_dev->info->read_raw(chan->indio_dev, chan->channel, > - val, &val2, 0); > + ret = chan->indio_dev->info->read_raw(chan->indio_dev, > + chan->channel, > + val, &val2, > + IIO_CHAN_INFO_RAW); > err_unlock: > mutex_unlock(&chan->indio_dev->info_exist_lock); > >