Re: [PATCH v1 0/6] media: nxp: imx7-media-csi: Move to subdev active state

From: Adam Ford <aford173@gmail.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: linux-media@vger.kernel.org, Rui Miguel Silva <rmfrfs@gmail.com>,
	Paul Elder <paul.elder@ideasonboard.com>,
	Martin Kepplinger <martin.kepplinger@puri.sm>,
	kernel@pengutronix.de, linux-imx@nxp.com
Subject: Re: [PATCH v1 0/6] media: nxp: imx7-media-csi: Move to subdev active state
Date: Thu, 26 Jan 2023 20:41:17 -0600	[thread overview]
Message-ID: <CAHCN7xKGfP81qvM_8rH299+HH_sj4dfhHODzG_Wf7eQWQjRhVA@mail.gmail.com> (raw)
In-Reply-To: <20230127022715.27234-1-laurent.pinchart@ideasonboard.com>

On Thu, Jan 26, 2023 at 8:27 PM Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
>
> Hello,
>
> This small series moves the imx7-mipi-csi driver to use the subdev
> active state. Patches 1/6 to 5/6 are small preparatory cleanups, with
> the main change in 6/6.
>
> I haven't tested the series yet as I need to dig up the hardware first.
> Adam, you offered to test the similar imx-mipi-csis series I've sent
> recently on the i.MX8MM, would you be able to test this one at the same
> time ?

Yes.  I'll try to do it tonight and get you feedback.  I'm out of town
for the next few days, so if I don't respond to follow up stuff right
away, it's because I am away.

adam
>
> Laurent Pinchart (6):
>   media: imx: imx7-media-csi: Drop imx7_csi.cc field
>   media: imx: imx7-media-csi: Simplify imx7_csi_video_init_format()
>   media: imx: imx7-media-csi: Drop unneeded check when starting
>     streaming
>   media: imx: imx7-media-csi: Drop unneeded src_sd check
>   media: imx: imx7-media-csi: Drop unneeded pad checks
>   media: imx: imx7-media-csi: Use V4L2 subdev active state
>
>  drivers/media/platform/nxp/imx7-media-csi.c | 208 ++++++--------------
>  1 file changed, 58 insertions(+), 150 deletions(-)
>
>
> base-commit: 1b929c02afd37871d5afb9d498426f83432e71c2
> --
> Regards,
>
> Laurent Pinchart
>