Re: [PATCH 64/65] media: staging: ipu7: isys: Don't set V4L2_FL_USES_V4L2_FH manually

Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/[email protected]> Thu, 7 Aug 2025 20:01:35 +0300
Newsgroups gmane.linux.ports.tegra,gmane.linux.drivers.video-input-infrastructure,gmane.linux.kernel,gmane.linux.documentation,gmane.linux.ports.arm.kernel,gmane.linux.ports.arm.mediatek,gmane.linux.kernel.renesas-soc,gmane.linux.ports.arm.msm,gmane.linux.kernel.samsung-soc,gmane.linux.usb.general,gmane.linux.ports.arm.rockchip,gmane.comp.video.mjpeg.user
Message-ID <[email protected]>
Hi Sakari,

On Thu, Aug 07, 2025 at 09:07:13AM +0000, Sakari Ailus wrote:
> On Sat, Aug 02, 2025 at 11:23:26AM +0200, Jacopo Mondi wrote:
> > From: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/[email protected]>
> > 
> > The V4L2_FL_USES_V4L2_FH flag is set by v4l2_fh_init(). It is not meant
> > to be set manually by drivers. Drop it from the ipu7-isys driver.
> > 
> > Signed-off-by: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/[email protected]>
> > Signed-off-by: Jacopo Mondi <jacopo.mondi-ryLnwIuWjnjg/[email protected]>
> 
> Thanks, Jacopo!
> 
> Reviewed-by: Sakari Ailus <sakari.ailus-VuQAYsv1563Yd54FQh9/[email protected]>

Can we get you review on patch 63/65 too (for the IPU6 driver) too ?

-- 
Regards,

Laurent Pinchart