Hi Laurent,
Thanks for extending support for imx8.
On Sun May 16, 2021 at 3:42 AM WEST, Laurent Pinchart wrote:
> Hello,
>
> This small patch series updates the imx7-media-csi driver to work on the
> i.MX8MM with an OV5640 sensor.
>
> Patch 1/3 extends the nxp,imx7-csi DT bindings with a compatible string
> for the i.MX8MM. While the CSI bridge in that SoC doesn't seem to differ
> from the one in the i.MX7 according to the reference manual, experience
> shows that NXP reference manuals are not always reliable. To be on the
> safe side, a new fsl,imx8mm-csi compatible string, with a fallback on
> fsl,imx7-csi, will avoid future backward-compatibility problems.
>
> Patches 2/3 and 3/3 fix issues with RAW8 and RAW10 capture from an
> OV5640 sensor. The fixes are the result of experimentation and study of
> NXP BSP drivers, as the reference manual doesn't provide much
> information in this area. I'm not very happy with this, as understanding
> the exact effect of the register fields modified by those two patches
> would be better. Still, without support from NXP (which I would really,
> really appreciate - anyone from NXP reading this ?), I can't do better.
>
> Given those concerns, I would also appreciate if this series could be
> tested widely for possible regressions. There should be no change for
> YUV formats, so only raw formats (RAW8, RAW10, RAW12 and RAW14) need to
> be tested.
I've tested with my setup in imx7 which is RAW10 only, and everything
looks fine.
I only have a small suggestion in 2/3.
------
Cheers,
Rui
>
> Laurent Pinchart (3):
> dt-bindings: media: nxp,imx7-csi: Add i.MX8MM support
> media: imx: imx7-media-csi: Set TWO_8BIT_SENSOR for >= 10-bit formats
> media: imx: imx7-media-csi: Don't set PIXEL_BIT in CSICR1
>
> .../bindings/media/nxp,imx7-csi.yaml | 12 +++++++----
> drivers/staging/media/imx/imx7-media-csi.c | 21 +++++--------------
> 2 files changed, 13 insertions(+), 20 deletions(-)
>
> --
> Regards,
>
> Laurent Pinchart
