Hi Christophe, .... > + > + data = iio_priv(indio_dev); > + dev_set_drvdata(dev, indio_dev); > + data->regmap = regmap; > + data->capture = 0; No need to explicitly initialize 'capture', devm_iio_device_alloc() already zeroes the allocated emmory. It doesn't hurt to be explicit, but why this field and not the other ones? -> This is the flag to enable capture mode. It is important to be disabled by default, therefore rather make this explicitly. ... Jianping
