On Sun, Mar 13, 2022 at 12:18:39AM +0200, Sakari Ailus wrote: > > @@ -1193,7 +1189,7 @@ static int mipi_csis_log_status(struct v4l2_subdev *sd) > > > > mutex_lock(&csis->lock); > > mipi_csis_log_counters(csis, true); > > - if (csis->debug.enable && (csis->state & ST_POWERED)) > > + if (csis->debug.enable && csis->powered) > > Could you use instead pm_runtime_get_if_active() instead? You could then > drop the field. Ah, I see Jacopo already posted a patch to do that. Feel free to ignore. -- Sakari Ailus