On Wed, Apr 27, 2022 at 12:57:16PM +0300, Sakari Ailus wrote: > > + ret = pm_runtime_resume_and_get(&client->dev); > > Please see how e.g. the CCS driver does this (no need for autosuspend > though). E.g. don't use runtime PM to power the sensor on in probe, or off > in remove. What's actually needed in this driver would be explicit use of the resume and suspend callbacks for !CONFIG_PM as well as setting the callbacks in ov5670_pm_ops. No other changes should be needed. -- Sakari Ailus