On 08/02/16 15:03, Daniel Baluta wrote: > mlock *must* be used by core and drivers to protect access > to devices state changes. > > Signed-off-by: Daniel Baluta <daniel.baluta@xxxxxxxxx> Applied. Thanks for cleaning this up. Jonathan > --- > At a first glance there are few places where access to iio_dev > state changes are done without mlock protection. Will fix them > with follow up patches. > > include/linux/iio/iio.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/iio/iio.h b/include/linux/iio/iio.h > index b589411..ce9e9c1 100644 > --- a/include/linux/iio/iio.h > +++ b/include/linux/iio/iio.h > @@ -448,7 +448,7 @@ struct iio_buffer_setup_ops { > * @buffer: [DRIVER] any buffer present > * @buffer_list: [INTERN] list of all buffers currently attached > * @scan_bytes: [INTERN] num bytes captured to be fed to buffer demux > - * @mlock: [INTERN] lock used to prevent simultaneous device state > + * @mlock: [DRIVER] lock used to prevent simultaneous device state > * changes > * @available_scan_masks: [DRIVER] optional array of allowed bitmasks > * @masklength: [INTERN] the length of the mask established from > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html