Re: [PATCH v3 2/2] iio: light: veml3328: add support for new device

[Jonathan Cameron]

On Sun, 31 May 2026 18:44:43 +0200
Joshua Crofts <joshua.crofts1@xxxxxxxxx> wrote:

> On Sun, 31 May 2026 at 11:19, Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> > > +     pm_runtime_set_active(dev);
> > > +     pm_runtime_set_autosuspend_delay(dev, 2000);
> > > +     pm_runtime_use_autosuspend(dev);  
> >
> > As per other thread check this results in power down.
> > If it doesn't, try moving autosupend calls after devm_pm_runtime_enable()
> > and see if that is enough.  Otherwise, you can either manipulate the counters
> > or I think just call pm_runtime_idle() to force it off immediately.  
> 
> Just to confirm, I've tested this on the actual hardware and it did autosuspend
> after probe, so I'm going to say that Sashiko is a bit off for once.

If you have time to figure out the path that causes that, it would be excellent
to have it recorded. I spent a few minutes the other day on this and couldn't
find it.  Maybe ask AI to figure it out :)

J
>