Re: [PATCH v5 1/2] media: imx8-isi: crossbar: Add get_frame_desc operation

[Laurent Pinchart]

Hi Guoniu,

Thank you for the patch.

On Thu, May 21, 2026 at 05:10:04PM +0800, Guoniu Zhou wrote:
> From: "Guoniu.zhou" <guoniu.zhou@xxxxxxx>
> 
> Implement the get_frame_desc pad operation for the crossbar subdev using
> the v4l2_subdev_get_frame_desc_passthrough() helper. This allows the
> crossbar to properly propagate frame descriptors from its sink pads to
> its source pads, which is necessary for proper stream configuration in
> multiplexed streams scenarios.
> 
> Signed-off-by: Guoniu.zhou <guoniu.zhou@xxxxxxx>

Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>

> ---
> Changes in v5:
> - Use v4l2_subdev_get_frame_desc_passthrough helper
> - Rewrote commit message
> 
> Changes in v4:
> - Use %d instead of %u for ret variable in error messages
> - Fix potential -ENOIOCTLCMD leak by resetting ret to 0 on continue
> 
> Changes in v3:
> - New patch added based on feedback from Laurent Pinchart
> ---
>  drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c b/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c
> index 605a45124103..0b593aed618b 100644
> --- a/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c
> +++ b/drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c
> @@ -404,6 +404,7 @@ static const struct v4l2_subdev_pad_ops mxc_isi_crossbar_subdev_pad_ops = {
>  	.enum_mbus_code = mxc_isi_crossbar_enum_mbus_code,
>  	.get_fmt = v4l2_subdev_get_fmt,
>  	.set_fmt = mxc_isi_crossbar_set_fmt,
> +	.get_frame_desc = v4l2_subdev_get_frame_desc_passthrough,
>  	.set_routing = mxc_isi_crossbar_set_routing,
>  	.enable_streams = mxc_isi_crossbar_enable_streams,
>  	.disable_streams = mxc_isi_crossbar_disable_streams,

-- 
Regards,

Laurent Pinchart