From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:46812 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965365AbaLLKVP (ORCPT ); Fri, 12 Dec 2014 05:21:15 -0500 Message-ID: <548AC199.7060009@kernel.org> Date: Fri, 12 Dec 2014 10:21:13 +0000 From: Jonathan Cameron MIME-Version: 1.0 To: Daniel Baluta , Lars-Peter Clausen CC: Hartmut Knaack , Peter Meerwald , "linux-iio@vger.kernel.org" Subject: Re: [PATCH 01/11] staging:iio:ad5933: Don't enable channels by default References: <1417024517-7564-1-git-send-email-lars@metafoo.de> <1417024517-7564-2-git-send-email-lars@metafoo.de> In-Reply-To: Content-Type: text/plain; charset=utf-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 04/12/14 22:51, Daniel Baluta wrote: > On Wed, Nov 26, 2014 at 7:55 PM, Lars-Peter Clausen wrote: >> The convention for IIO devices is that all channels are disabled by default. To be fair - it wasn't the convention when these were written ;) >> >> Signed-off-by: Lars-Peter Clausen > > Reviewed-by: Daniel Baluta Applied to the togreg branch of iio.git. It's just much easier to ensure their are no bugs and everything is predictable. This way we don't end up with people relying on a particular set of channels being enabled. > >> --- >> drivers/staging/iio/impedance-analyzer/ad5933.c | 4 ---- >> 1 file changed, 4 deletions(-) >> >> diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c >> index b6bd609..aa6a368 100644 >> --- a/drivers/staging/iio/impedance-analyzer/ad5933.c >> +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c >> @@ -757,10 +757,6 @@ static int ad5933_probe(struct i2c_client *client, >> if (ret) >> goto error_unreg_ring; >> >> - /* enable both REAL and IMAG channels by default */ >> - iio_scan_mask_set(indio_dev, indio_dev->buffer, 0); >> - iio_scan_mask_set(indio_dev, indio_dev->buffer, 1); >> - >> ret = ad5933_setup(st); >> if (ret) >> goto error_uninitialize_ring; >> -- >> 1.8.0 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html