Re: [PATCH v4 00/13] media: imx8: add camera support

[Frank Li <Frank.li@nxp.com>]

On Tue, Apr 08, 2025 at 05:52:58PM -0400, Frank Li wrote:
> Add SCU reset driver for i.MX8QM/i.MX8QXP.
> Update binding doc.
> Update driver for imx8qxp and imx8qm.
> Add dts files for it.
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Laurent Pinchart:

	Do you have chance to check this version?

Frank Li

>
> Changes in v4:
> - Add 4 clean up patches
> 	media: nxp: imx8-isi: Remove unused offset in mxc_isi_reg and use BIT() macro for mask
> 	media: nxp: imx8-isi: Use dev_err_probe() simplify code
> 	media: nxp: imx8-isi: Remove redundant check for dma_set_mask_and_coherent()
> 	media: nxp: imx8-isi: Use devm_clk_bulk_get_all() to fetch clocks
> - rebase to v6.15-rc1.
> - Remove scu reset patches, which already in linux-next
> - Remove patch
> 	 Add fixed clock node clock-xtal24m to prepare to add camera support.
> - other detail change log see each patch's change log
> - Link to v3: https://lore.kernel.org/r/20250210-8qxp_camera-v3-0-324f5105accc@nxp.com
>
> Changes in v3:
> - Remove phy driver parts.
> - csr is dedicate for mipi csi2, so add it as second register space. csr is
> mixed with PHY and link control with csi2.
> - Link to v2: https://lore.kernel.org/r/20250205-8qxp_camera-v2-0-731a3edf2744@nxp.com
>
> Changes in v2:
> - move scu reset binding doc to top scu doc.
> - isi use seperate binding doc for imx8qxp and imx8qm.
> - phy and csi2, compatible string 8qm fallback to qxp
> - remove internal review tags
> - Link to v1: https://lore.kernel.org/r/20250131-8qxp_camera-v1-0-319402ab606a@nxp.com
>
> ---
> Frank Li (10):
>       media: dt-bindings: Add binding doc for i.MX8QXP and i.MX8QM ISI
>       media: nxp: imx8-isi: Allow num_sources to be greater than num_sink
>       media: nxp: imx8-isi: Remove unused offset in mxc_isi_reg and use BIT() macro for mask
>       media: nxp: imx8-isi: Use devm_clk_bulk_get_all() to fetch clocks
>       media: nxp: imx8-isi: Remove redundant check for dma_set_mask_and_coherent()
>       media: nxp: imx8-isi: Use dev_err_probe() simplify code
>       media: imx8mq-mipi-csi2: Add support for i.MX8QXP
>       arm64: dts: imx8: add capture controller for i.MX8's img subsystem
>       arm64: dts: imx8q: add linux,cma node for imx8qm-mek and imx8qxp-mek
>       arm64: dts: imx8q: add camera ov5640 support for imx8qm-mek and imx8qxp-mek
>
> Guoniu.zhou (1):
>       media: imx8mq-mipi-csi2: Add imx8mq_plat_data for different compatible strings
>
> Robert Chiras (2):
>       media: imx8-isi: Add support for i.MX8QM and i.MX8QXP
>       media: dt-bindings: nxp,imx8mq-mipi-csi2: Add i.MX8QM(QXP) compatible strings
>
>  .../devicetree/bindings/media/fsl,imx8qm-isi.yaml  | 117 +++++++
>  .../devicetree/bindings/media/fsl,imx8qxp-isi.yaml | 106 ++++++
>  .../bindings/media/nxp,imx8mq-mipi-csi2.yaml       |  38 ++-
>  MAINTAINERS                                        |   1 +
>  arch/arm64/boot/dts/freescale/Makefile             |  11 +
>  arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi     | 362 +++++++++++++++++++++
>  .../boot/dts/freescale/imx8qm-mek-ov5640-csi0.dtso |  60 ++++
>  .../boot/dts/freescale/imx8qm-mek-ov5640-csi1.dtso |  60 ++++
>  arch/arm64/boot/dts/freescale/imx8qm-mek.dts       |  67 ++++
>  arch/arm64/boot/dts/freescale/imx8qm-ss-img.dtsi   |  85 +++++
>  arch/arm64/boot/dts/freescale/imx8qm.dtsi          |   5 +
>  .../boot/dts/freescale/imx8qxp-mek-ov5640-csi.dtso |  59 ++++
>  arch/arm64/boot/dts/freescale/imx8qxp-mek.dts      |  44 +++
>  arch/arm64/boot/dts/freescale/imx8qxp-ss-img.dtsi  |  86 +++++
>  arch/arm64/boot/dts/freescale/imx8qxp.dtsi         |   5 +
>  .../media/platform/nxp/imx8-isi/imx8-isi-core.c    | 133 ++++----
>  .../media/platform/nxp/imx8-isi/imx8-isi-core.h    |   6 +-
>  .../platform/nxp/imx8-isi/imx8-isi-crossbar.c      |   8 +-
>  drivers/media/platform/nxp/imx8mq-mipi-csi2.c      | 169 +++++++++-
>  19 files changed, 1328 insertions(+), 94 deletions(-)
> ---
> base-commit: 1f665976a7c4e8779566e153b8854d7829ce33ac
> change-id: 20250114-8qxp_camera-c1af5749d304
>
> Best regards,
> ---
> Frank Li <Frank.Li@nxp.com>
>