From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH] Staging: IIO: add selection of IIO_SW_RING to LIS3L02DQ as needed Date: Mon, 07 Dec 2009 09:07:53 -0800 Message-ID: <4B1D3669.4020204@oracle.com> References: <20091204203014.e8ee54ca.sfr@canb.auug.org.au> <20091206101225.7d8ce4a4.randy.dunlap@oracle.com> <4B1D1CF5.6030502@cam.ac.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from acsinet12.oracle.com ([141.146.126.234]:45637 "EHLO acsinet12.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757954AbZLGRJH (ORCPT ); Mon, 7 Dec 2009 12:09:07 -0500 In-Reply-To: <4B1D1CF5.6030502@cam.ac.uk> Sender: linux-next-owner@vger.kernel.org List-ID: To: Jonathan Cameron Cc: Randy Dunlap , Stephen Rothwell , gregkh@suse.de, linux-next@vger.kernel.org, LKML Jonathan Cameron wrote: > Signed-off-by: Jonathan Cameron > Reported-by: Randy Dunlap Acked-by: Randy Dunlap Thanks. > --- > > Thanks to Randy for the report. Sorry for the slow response. > I've been meaning to carefully check for any remaining cases like > this. > Here I've kept the selection of IIO_SW_RING separate from > IIO_TRIGGER as it will go away fairly shortly when the ring buffer > type becomes configurable on a per device basis, whereas the > IIO_TRIGGER select will remain. Whether to retain the option to > remove the support for ring buffers entirely is one for after that > support is in place. > > Touch wood I think this is the last one, but I'll try a selection of > unusual combinations now and post others as replies to this email. > > drivers/staging/iio/accel/Kconfig | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/staging/iio/accel/Kconfig b/drivers/staging/iio/accel/Kconfig > index 4044663..3d3c333 100644 > --- a/drivers/staging/iio/accel/Kconfig > +++ b/drivers/staging/iio/accel/Kconfig > @@ -14,6 +14,7 @@ config LIS3L02DQ > tristate "ST Microelectronics LIS3L02DQ Accelerometer Driver" > depends on SPI > select IIO_TRIGGER if IIO_RING_BUFFER > + select IIO_SW_RING if IIO_RING_BUFFER > help > Say yes here to build SPI support for the ST microelectronics > accelerometer. The driver supplies direct access via sysfs files -- ~Randy