Re: [Outreachy kernel] Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-19 Thread Gargi Sharma
On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: > On 03/12/2017 02:32 PM, simran singhal wrote: >> The IIO subsystem is redefining iio_dev->mlock to be used by >> the IIO core only for protecting device operating mode changes. >> ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* mod

Re: [Outreachy kernel] Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-19 Thread Jonathan Cameron
On 19/03/17 13:16, Gargi Sharma wrote: > On Sun, Mar 19, 2017 at 4:01 PM, Jonathan Cameron wrote: >> On 17/03/17 09:32, Gargi Sharma wrote: >>> On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: On 03/12/2017 02:32 PM, simran singhal wrote: > The IIO subsystem is redefining

Re: [Outreachy kernel] Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-19 Thread Gargi Sharma
On Sun, Mar 19, 2017 at 4:01 PM, Jonathan Cameron wrote: > On 17/03/17 09:32, Gargi Sharma wrote: >> On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: >>> >>> On 03/12/2017 02:32 PM, simran singhal wrote: The IIO subsystem is redefining iio_dev->mlock to be used by the IIO core

Re: [Outreachy kernel] Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-19 Thread Jonathan Cameron
On 17/03/17 09:32, Gargi Sharma wrote: > On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: >> >> On 03/12/2017 02:32 PM, simran singhal wrote: >>> The IIO subsystem is redefining iio_dev->mlock to be used by >>> the IIO core only for protecting device operating mode changes. >>> ie. Chang

Re: [Outreachy kernel] Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-17 Thread Gargi Sharma
On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: > > On 03/12/2017 02:32 PM, simran singhal wrote: > > The IIO subsystem is redefining iio_dev->mlock to be used by > > the IIO core only for protecting device operating mode changes. > > ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_

Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-13 Thread Lars-Peter Clausen
On 03/13/2017 01:33 PM, SIMRAN SINGHAL wrote: > On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: >> On 03/12/2017 02:32 PM, simran singhal wrote: >>> The IIO subsystem is redefining iio_dev->mlock to be used by >>> the IIO core only for protecting device operating mode changes. >>> ie. C

Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-13 Thread SIMRAN SINGHAL
On Mon, Mar 13, 2017 at 5:30 PM, Lars-Peter Clausen wrote: > On 03/12/2017 02:32 PM, simran singhal wrote: >> The IIO subsystem is redefining iio_dev->mlock to be used by >> the IIO core only for protecting device operating mode changes. >> ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* mod

Re: [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-13 Thread Lars-Peter Clausen
On 03/12/2017 02:32 PM, simran singhal wrote: > The IIO subsystem is redefining iio_dev->mlock to be used by > the IIO core only for protecting device operating mode changes. > ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* modes. > > In this driver, mlock was being used to protect hardware

Re: [Outreachy kernel] [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-12 Thread Alison Schofield
On Mon, Mar 13, 2017 at 09:28:34AM +0530, SIMRAN SINGHAL wrote: > On Mon, Mar 13, 2017 at 12:03 AM, Alison Schofield > wrote: > > On Sun, Mar 12, 2017 at 07:02:50PM +0530, simran singhal wrote: > >> The IIO subsystem is redefining iio_dev->mlock to be used by > >> the IIO core only for protecting

Re: [Outreachy kernel] [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-12 Thread SIMRAN SINGHAL
On Mon, Mar 13, 2017 at 12:03 AM, Alison Schofield wrote: > On Sun, Mar 12, 2017 at 07:02:50PM +0530, simran singhal wrote: >> The IIO subsystem is redefining iio_dev->mlock to be used by >> the IIO core only for protecting device operating mode changes. >> ie. Changes between INDIO_DIRECT_MODE, I

Re: [Outreachy kernel] [PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-12 Thread Alison Schofield
On Sun, Mar 12, 2017 at 07:02:50PM +0530, simran singhal wrote: > The IIO subsystem is redefining iio_dev->mlock to be used by > the IIO core only for protecting device operating mode changes. > ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* modes. > > In this driver, mlock was being used t

[PATCH] staging: iio: ade7753: replace mlock with driver private lock

2017-03-12 Thread simran singhal
The IIO subsystem is redefining iio_dev->mlock to be used by the IIO core only for protecting device operating mode changes. ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* modes. In this driver, mlock was being used to protect hardware state changes. Replace it with a lock in the devices g