Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752548AbaAMVZN (ORCPT ); Mon, 13 Jan 2014 16:25:13 -0500 Received: from ns.pmeerw.net ([87.118.82.44]:48037 "EHLO pmeerw.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752060AbaAMVZL (ORCPT ); Mon, 13 Jan 2014 16:25:11 -0500 Date: Mon, 13 Jan 2014 22:25:10 +0100 (CET) From: Peter Meerwald To: Ivaylo Dimitrov cc: jic23@kernel.org, pali.rohar@gmail.com, pavel@ucw.cz, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Ivaylo Dimitrov Subject: Re: [PATCH] iio: tsl2563: Use the correct channel2 member In-Reply-To: <1389633859-5972-1-git-send-email-ivo.g.dimitrov.75@gmail.com> Message-ID: References: <52D30BF7.90107@kernel.org> <1389633859-5972-1-git-send-email-ivo.g.dimitrov.75@gmail.com> User-Agent: Alpine 2.01 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Use the correct channel2 member instead of channel when dealing with sysfs > reads/writes > Signed-off-by: Ivaylo Dimitrov as a bonus, m is renamed to mask Acked-by: Peter Meerwald > --- > drivers/iio/light/tsl2563.c | 16 ++++++++++------ > 1 files changed, 10 insertions(+), 6 deletions(-) > > diff --git a/drivers/iio/light/tsl2563.c b/drivers/iio/light/tsl2563.c > index 5e5d9de..2be6df3 100644 > --- a/drivers/iio/light/tsl2563.c > +++ b/drivers/iio/light/tsl2563.c > @@ -460,10 +460,14 @@ static int tsl2563_write_raw(struct iio_dev *indio_dev, > { > struct tsl2563_chip *chip = iio_priv(indio_dev); > > - if (chan->channel == IIO_MOD_LIGHT_BOTH) > + if (mask != IIO_CHAN_INFO_CALIBSCALE) > + return -EINVAL; > + if (chan->channel2 == IIO_MOD_LIGHT_BOTH) > chip->calib0 = calib_from_sysfs(val); > - else > + else if (chan->channel2 == IIO_MOD_LIGHT_IR) > chip->calib1 = calib_from_sysfs(val); > + else > + return -EINVAL; > > return 0; > } > @@ -472,14 +476,14 @@ static int tsl2563_read_raw(struct iio_dev *indio_dev, > struct iio_chan_spec const *chan, > int *val, > int *val2, > - long m) > + long mask) > { > int ret = -EINVAL; > u32 calib0, calib1; > struct tsl2563_chip *chip = iio_priv(indio_dev); > > mutex_lock(&chip->lock); > - switch (m) { > + switch (mask) { > case IIO_CHAN_INFO_RAW: > case IIO_CHAN_INFO_PROCESSED: > switch (chan->type) { > @@ -498,7 +502,7 @@ static int tsl2563_read_raw(struct iio_dev *indio_dev, > ret = tsl2563_get_adc(chip); > if (ret) > goto error_ret; > - if (chan->channel == 0) > + if (chan->channel2 == IIO_MOD_LIGHT_BOTH) > *val = chip->data0; > else > *val = chip->data1; > @@ -510,7 +514,7 @@ static int tsl2563_read_raw(struct iio_dev *indio_dev, > break; > > case IIO_CHAN_INFO_CALIBSCALE: > - if (chan->channel == 0) > + if (chan->channel2 == IIO_MOD_LIGHT_BOTH) > *val = calib_to_sysfs(chip->calib0); > else > *val = calib_to_sysfs(chip->calib1); > -- Peter Meerwald +43-664-2444418 (mobile) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/