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

2017-03-19 Thread Jonathan Cameron
On 18/03/17 18:46, SIMRAN SINGHAL wrote: > On Sat, Mar 18, 2017 at 11:51 PM, Jonathan Cameron wrote: >> On 18/03/17 17:34, SIMRAN SINGHAL wrote: >>> On Thu, Mar 16, 2017 at 3:23 AM, Jonathan Cameron wrote: On 13/03/17 19:53, Alison Schofield wrote: > On Mon, Mar 13, 2017 at 10:01:07PM +0

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

2017-03-18 Thread SIMRAN SINGHAL
On Sat, Mar 18, 2017 at 11:51 PM, Jonathan Cameron wrote: > On 18/03/17 17:34, SIMRAN SINGHAL wrote: >> On Thu, Mar 16, 2017 at 3:23 AM, Jonathan Cameron wrote: >>> On 13/03/17 19:53, Alison Schofield wrote: On Mon, Mar 13, 2017 at 10:01:07PM +0530, simran singhal wrote: > The IIO subsys

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

2017-03-18 Thread Jonathan Cameron
On 18/03/17 17:34, SIMRAN SINGHAL wrote: > On Thu, Mar 16, 2017 at 3:23 AM, Jonathan Cameron wrote: >> On 13/03/17 19:53, Alison Schofield wrote: >>> On Mon, Mar 13, 2017 at 10:01:07PM +0530, simran singhal wrote: The IIO subsystem is redefining iio_dev->mlock to be used by the IIO core

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

2017-03-18 Thread SIMRAN SINGHAL
On Thu, Mar 16, 2017 at 3:23 AM, Jonathan Cameron wrote: > On 13/03/17 19:53, Alison Schofield wrote: >> On Mon, Mar 13, 2017 at 10:01:07PM +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 chang

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

2017-03-15 Thread Jonathan Cameron
On 13/03/17 19:53, Alison Schofield wrote: > On Mon, Mar 13, 2017 at 10:01:07PM +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_* mo

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

2017-03-13 Thread Alison Schofield
On Mon, Mar 13, 2017 at 10:01:07PM +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 v2] staging: iio: ade7753: replace mlock with driver private lock

2017-03-13 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