Re: [PATCH 1/1] v4l: fwnode: v4l2_async_notifier_parse_fwnode_endpoints is deprecated

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Hi Paul,

On Fri, Jan 08, 2021 at 10:57:57AM +0100, Paul Kocialkowski wrote:
> Hi,
> 
> On Fri 08 Jan 21, 00:07, Sakari Ailus wrote:
> > Hi Ezequiel,
> > 
> > On Thu, Jan 07, 2021 at 04:31:58PM -0300, Ezequiel Garcia wrote:
> > > Hi Sakari,
> > > 
> > > On Mon, 14 Dec 2020 at 18:45, Sakari Ailus <sakari.ailus@linux.intel.com> wrote:
> > > >
> > > > Document that v4l2_async_notifier_parse_fwnode_endpoints() is deprecated.
> > > > Its functionality has been replaced by other, better functions. Also add a
> > > > reference to an example if someone ends up wandering here.
> > > >
> > > 
> > > I'm working on a series to clean up the v4l2_async API a bit,
> > > and came across this patch.
> > > 
> > > As far as I can see, the only user of v4l2_async_notifier_parse_fwnode_endpoints
> > > is drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c.
> > 
> > It may be that's the only one left. The intent was also to avoid anyone
> > proposing new ones.
> > 
> > > Adding Maxime and Yong Deng, it would be great to get rid of this,
> > > so we can remove the API, which would also allow us to remove
> > > some other internal functions and therefore make the API a bit cleaner.
> > 
> > Yes, that'd be very nice. Then we could remove it altogether.
> 
> I've been touching that area a bit with my MIPI CSI-2 support series.
> I'll try to get rid of it along the way if I need to respin the series,
> or come up with a follow-up patch to remove the call to this function.
> 
> If I understand correctly, the preferred way now is to use:
> - fwnode_graph_get_endpoint_by_id
> - v4l2_fwnode_endpoint_parse
> - v4l2_async_notifier_add_fwnode_remote_subdev
> 
> Is that correct? I think that's what I've been doing in the MIPI CSI-2
> bridge drivers so far.

Yes, that's the preferred way.

-- 
Sakari Ailus