On Mon, 17 Oct 2022 09:30:01 +0300 Matti Vaittinen <mazziesaccount@xxxxxxxxx> wrote: > On 10/16/22 19:34, Jonathan Cameron wrote: > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > > > All the remaining calls in probe() have devm_ equivalents so > > switching to those allows the remove() callbacks to be deleted. > > No functional change. > > Right :) So please ignore my last comment in the patch 13/14 :) The one > concerning the commit message is still valid though :) > > > > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Cc: Tomasz Duszynski <tduszyns@xxxxxxxxx> > > Reviewed-by: Matti Vaittinen <mazziesaccount@xxxxxxxxx> Applied to the togreg branch of iio.git - some fuzz due to crossing with i2c probe_new() mass conversion and the changes that affected the previous patch. If anyone wants to sanity check the result, see the testing branch of iio.git in a few minutes. Thanks, Jonathan > > > --- > > drivers/iio/pressure/ms5611.h | 1 - > > drivers/iio/pressure/ms5611_core.c | 17 +++-------------- > > drivers/iio/pressure/ms5611_i2c.c | 6 ------ > > drivers/iio/pressure/ms5611_spi.c | 6 ------ > > 4 files changed, 3 insertions(+), 27 deletions(-) > > >