Re: [PATCH 14/14] iio: pressure: ms5611: Switch to fully devm_ managed registration.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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(-)
> >   
> 




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux