Re: [PATCH 1/8] media: ov08x40: Properly turn sensor on/off when runtime-suspended

["Bryan O'Donoghue" <bryan.odonoghue@xxxxxxxxxx>]

On 20/12/2024 13:47, Hans de Goede wrote:
> > >    }
> > >    +static DEFINE_RUNTIME_DEV_PM_OPS(ov08x40_pm_ops, ov08x40_power_on,
> > > +                 ov08x40_power_off, NULL);
> > > +
> Ugh I have on/off swapped here, second argument of the macro is the suspend
> function so that should be ov08x40_power_off. IOW this should be:
>
> static DEFINE_RUNTIME_DEV_PM_OPS(ov08x40_pm_ops, ov08x40_power_off,
> 				 ov08x40_power_on, NULL);
>
> Can you give this a try with that change?
>
> Regards,
>
> Hans

Heh.

That'd do it, works now.

---
bod