Re: [PATCH] iio: accel: kxcjk-1013: disable autosuspend on remove

[Jonathan Cameron <jic23@kernel.org>]

On Mon, 14 Sep 2026 14:49:20 +0200
Joshua Crofts <joshua.crofts1@gmail.com> wrote:

> On Mon, 14 Sep 2026 19:20:53 +0800
> Guangshuo Li <lgs201920130244@gmail.com> wrote:
> 
> ...
> 
> > Add the missing pm_runtime_dont_use_autosuspend() call to the remove
> > path.
> >
> > This issue was found by manual code inspection.
> >  
> 
> I'm on the fence whether this sentence should go under the ---
In this world of LLMs I'm going to stick up for human effort and
keep that in the commit message :)

Either would have been fine with me.

I'm not going to rush this one in, so applied to the testing branch of
iio.git

Thanks,

Jonathan

>  
> ...
> 
> > @@ -1547,6 +1547,7 @@ static void kxcjk1013_remove(struct i2c_client *client)
> >  
> >  	iio_device_unregister(indio_dev);
> >  
> > +	pm_runtime_dont_use_autosuspend(&client->dev);
> >  	pm_runtime_disable(&client->dev);
> >  	pm_runtime_set_suspended(&client->dev);
> >    
> 
> It would've been better if you sent the 4 patches as a series with a cover
> letter as all of them are the same fix.
> 
> Reviewed-by: Joshua Crofts <joshua.crofts1@gmail.com>
>