Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754534AbcC1Jv2 (ORCPT ); Mon, 28 Mar 2016 05:51:28 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:44517 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753327AbcC1JvZ (ORCPT ); Mon, 28 Mar 2016 05:51:25 -0400 Subject: Re: [PATCH 1/6] iio: accel: bmc150: use available_scan_masks To: Irina Tirdea , linux-iio@vger.kernel.org References: <1458811771-25217-1-git-send-email-irina.tirdea@intel.com> <1458811771-25217-2-git-send-email-irina.tirdea@intel.com> Cc: linux-kernel@vger.kernel.org, Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , Octavian Purdila , Markus Pargmann , Srinivas Pandruvada From: Jonathan Cameron Message-ID: <56F8FE98.8000203@kernel.org> Date: Mon, 28 Mar 2016 10:51:20 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1458811771-25217-2-git-send-email-irina.tirdea@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 Content-Length: 1990 Lines: 56 On 24/03/16 09:29, Irina Tirdea wrote: > Use available_scan_masks to allow the iio core to select > the data to send to userspace depending on which axes are > enabled, instead of doing this in the driver's interrupt > handler. > > Signed-off-by: Irina Tirdea Applied to the togreg branch of iio.git. Thanks, > --- > drivers/iio/accel/bmc150-accel-core.c | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/drivers/iio/accel/bmc150-accel-core.c b/drivers/iio/accel/bmc150-accel-core.c > index c73331f7..cc52366 100644 > --- a/drivers/iio/accel/bmc150-accel-core.c > +++ b/drivers/iio/accel/bmc150-accel-core.c > @@ -138,6 +138,7 @@ enum bmc150_accel_axis { > AXIS_X, > AXIS_Y, > AXIS_Z, > + AXIS_MAX, > }; > > enum bmc150_power_modes { > @@ -1104,6 +1105,10 @@ static const struct iio_info bmc150_accel_info_fifo = { > .driver_module = THIS_MODULE, > }; > > +static const unsigned long bmc150_accel_scan_masks[] = { > + BIT(AXIS_X) | BIT(AXIS_Y) | BIT(AXIS_Z), > + 0}; > + > static irqreturn_t bmc150_accel_trigger_handler(int irq, void *p) > { > struct iio_poll_func *pf = p; > @@ -1113,8 +1118,7 @@ static irqreturn_t bmc150_accel_trigger_handler(int irq, void *p) > unsigned int raw_val; > > mutex_lock(&data->mutex); > - for_each_set_bit(bit, indio_dev->active_scan_mask, > - indio_dev->masklength) { > + for (bit = 0; bit < AXIS_MAX; bit++) { > ret = regmap_bulk_read(data->regmap, > BMC150_ACCEL_AXIS_TO_REG(bit), &raw_val, > 2); > @@ -1574,6 +1578,7 @@ int bmc150_accel_core_probe(struct device *dev, struct regmap *regmap, int irq, > indio_dev->channels = data->chip_info->channels; > indio_dev->num_channels = data->chip_info->num_channels; > indio_dev->name = name ? name : data->chip_info->name; > + indio_dev->available_scan_masks = bmc150_accel_scan_masks; > indio_dev->modes = INDIO_DIRECT_MODE; > indio_dev->info = &bmc150_accel_info; > >