On Fri, Aug 23, 2024 at 07:16:42AM +0000, Sakari Ailus wrote:
> On Fri, Aug 23, 2024 at 12:41:25AM +0300, Laurent Pinchart wrote:
> > Commit b97213a41140 ("media: v4l2-mc: Make v4l2_pipeline_pm_{get,put}
> > deprecated") marked the v4l2_pipeline_pm_get() and
> > v4l2_pipeline_pm_put() functions as deprecated, but forgot to address
> > the related v4l2_pipeline_link_notify() function similarly. Fix it.
> >
> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
>
> Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
>
> How about adding a warning for the use of these functions? Possibly on
> debug level if pr_warn_once() is considered too drastic?
I think we need to do a bit of homework first, as there's a large number
of drivers using these, directly or indirectly. We should at least
convert the sensor drivers still using .s_power() to runtime PM, to make
it possible to convert the other drivers.
> > ---
> > include/media/v4l2-mc.h | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/include/media/v4l2-mc.h b/include/media/v4l2-mc.h
> > index ed0a44b6eada..1837c9fd78cf 100644
> > --- a/include/media/v4l2-mc.h
> > +++ b/include/media/v4l2-mc.h
> > @@ -178,6 +178,9 @@ void v4l2_pipeline_pm_put(struct media_entity *entity);
> > * @flags: New link flags that will be applied
> > * @notification: The link's state change notification type (MEDIA_DEV_NOTIFY_*)
> > *
> > + * THIS FUNCTION IS DEPRECATED. DO NOT USE IN NEW DRIVERS. USE RUNTIME PM
> > + * ON SUB-DEVICE DRIVERS INSTEAD.
> > + *
> > * React to link management on powered pipelines by updating the use count of
> > * all entities in the source and sink sides of the link. Entities are powered
> > * on or off accordingly. The use of this function should be paired
> >
> > base-commit: a043ea54bbb975ca9239c69fd17f430488d33522
--
Regards,
Laurent Pinchart
