Hi Hans,
On Thu, Mar 27, 2025 at 05:55:29PM +0100, Hans de Goede wrote:
When v4l2_subdev_init_finalize() fails no changes have been made to the runtime-pm device state yet, so the probe_error_media_entity_cleanup rollback path should not touch the runtime-pm device state.
Instead this should be done from the probe_error_v4l2_subdev_cleanup rollback path. Note the pm_runtime_xxx() calls are put above the v4l2_subdev_cleanup() call to have the reverse call order of probe().
Fixes: 289c25923ecd ("media: ov2740: Use sub-device active state") Cc: stable@vger.kernel.org Reviewed-by: Bingbu Cao bingbu.cao@intel.com Signed-off-by: Hans de Goede hdegoede@redhat.com
Changes in v2:
- Add Fixes: tag
Usually there's no need to resend just for the tags. I've got v1 already in the tree so I'll mark this as "superseded".