Hi Adam, On Thu, Jan 26, 2023 at 07:54:10PM -0600, Adam Ford wrote: > On Thu, Jan 26, 2023 at 3:34 PM Laurent Pinchart wrote: > > > > Hello, > > > > This small series moves the imx-mipi-csis driver to use the subdev > > active state. Patches 1/5 to 3/5 are small preparatory cleanups, with > > the main change in 4/5. Patch 5/5 is further cleanup that could have > > been included in 4/5, but that should be easier to review as a separate > > patch. > > > > The series has been tested on the i.MX8MP with the ISI, and IMX296 and > > IMX327 camera sensors. > > I didn't notice any overall change in the CSIS capture on the imx8mn. > I can test the Mini if you want. That would be great. Would you be able to test it in conjunction with the similar imx7-media-csi series I've just sent ([1]) ? I expect a possible lockdep warning if this series is applied with the corresponding change in imx7-media-csi, but together they should be fine. [1] https://lore.kernel.org/linux-media/20230127022715.27234-1-laurent.pinchart@xxxxxxxxxxxxxxxx > For the series: > Tested-by: Adam Ford <aford173@xxxxxxxxx> #imx8mn-beacon > > > Laurent Pinchart (5): > > media: imx-mipi-csis: Rename error labels with 'err_' prefix > > media: imx-mipi-csis: Don't take lock in runtime PM handlers > > media: imx-mipi-csis: Pass format explicitly to internal functions > > media: imx-mipi-csis: Use V4L2 subdev active state > > media: imx-mipi-csis: Implement .init_cfg() using .set_fmt() > > > > drivers/media/platform/nxp/imx-mipi-csis.c | 249 +++++++++------------ > > 1 file changed, 103 insertions(+), 146 deletions(-) -- Regards, Laurent Pinchart
