From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755316AbbDINSx (ORCPT ); Thu, 9 Apr 2015 09:18:53 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:52443 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753235AbbDINSv (ORCPT ); Thu, 9 Apr 2015 09:18:51 -0400 Message-ID: <55267C39.5050707@kernel.org> Date: Thu, 09 Apr 2015 14:18:49 +0100 From: Jonathan Cameron User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Jacob Pan , IIO CC: LKML , Lee Jones , Lars-Peter Clausen , Hartmut Knaack , Aaron Lu , Todd Brandt Subject: Re: [PATCH v2] iio/axp288_adc: add missing channel info mask References: <1428345500-27672-1-git-send-email-jacob.jun.pan@linux.intel.com> In-Reply-To: <1428345500-27672-1-git-send-email-jacob.jun.pan@linux.intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/04/15 19:38, Jacob Pan wrote: > Commit 65de7654d39c70c2b ("iio: iio: Fix iio_channel_read return if > channel havn't info") added a check for valid info masks. > > This patch adds missing channel info masks for all ADC channels. > Otherwise, iio_read_channel_raw() would return -EINVAL when called > by consumer drivers. > > Signed-off-by: Jacob Pan This makes me a little nervous. Why did we have the separate raw and processed paths in the read_raw function in the first place? Also this changes the exposed userspace ABI by dropping those processed attributes that already exist.. Jonathan > --- > drivers/iio/adc/axp288_adc.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c > index 08bcfb0..56008a8 100644 > --- a/drivers/iio/adc/axp288_adc.c > +++ b/drivers/iio/adc/axp288_adc.c > @@ -53,39 +53,42 @@ static const struct iio_chan_spec const axp288_adc_channels[] = { > .channel = 0, > .address = AXP288_TS_ADC_H, > .datasheet_name = "TS_PIN", > + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), > }, { > .indexed = 1, > .type = IIO_TEMP, > .channel = 1, > .address = AXP288_PMIC_ADC_H, > .datasheet_name = "PMIC_TEMP", > + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), > }, { > .indexed = 1, > .type = IIO_TEMP, > .channel = 2, > .address = AXP288_GP_ADC_H, > .datasheet_name = "GPADC", > + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), > }, { > .indexed = 1, > .type = IIO_CURRENT, > .channel = 3, > .address = AXP20X_BATT_CHRG_I_H, > .datasheet_name = "BATT_CHG_I", > - .info_mask_separate = BIT(IIO_CHAN_INFO_PROCESSED), > + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), > }, { > .indexed = 1, > .type = IIO_CURRENT, > .channel = 4, > .address = AXP20X_BATT_DISCHRG_I_H, > .datasheet_name = "BATT_DISCHRG_I", > - .info_mask_separate = BIT(IIO_CHAN_INFO_PROCESSED), > + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), > }, { > .indexed = 1, > .type = IIO_VOLTAGE, > .channel = 5, > .address = AXP20X_BATT_V_H, > .datasheet_name = "BATT_V", > - .info_mask_separate = BIT(IIO_CHAN_INFO_PROCESSED), > + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), > }, > }; > > @@ -151,9 +154,6 @@ static int axp288_adc_read_raw(struct iio_dev *indio_dev, > chan->address)) > dev_err(&indio_dev->dev, "TS pin restore\n"); > break; > - case IIO_CHAN_INFO_PROCESSED: > - ret = axp288_adc_read_channel(val, chan->address, info->regmap); > - break; > default: > ret = -EINVAL; > } >