Hi Sakari
On 12/18/23 12:09, Sakari Ailus wrote:
Hi Julien,
Thanks for the patch.
On Mon, Dec 18, 2023 at 10:56:00AM +0100, Julien Massot wrote:
On some subdev the fwnode is the device node and not the endpoint node.
Using the subdev fwnode doesn't allow to fetch the subdev
pad we are connected to when the remote subdev has multiple
output pads.
I'd change the commit message, endpoint-to-endpoint matching used to be the
preferred way to do things not long ago.
Fixes: 1fc3b37f34f69 ("media: v4l: cadence: Add Cadence MIPI-CSI2 RX driver")
I think the driver was originally fine but I missed making the below change
when changing async matching. Instead I think you should have here:
Fixes: 1029939b3782 ("media: v4l: async: Simplify async sub-device fwnode matching")
IOW applying your patch without 1029939b3782 will also break cdns-csi2rx
driver.
I just replaced the fix tag, and the commit message as suggested in V3.
Thank you,
Julien
