| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20250611142503.GB22730@pendragon.ideasonboard.com> Date: Wed, 11 Jun 2025 17:25:03 +0300 From: Laurent Pinchart <laurent.pinchart@...asonboard.com> To: Frank Li <Frank.li@....com> Cc: Mauro Carvalho Chehab <mchehab@...nel.org>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>, Fabio Estevam <festevam@...il.com>, Rui Miguel Silva <rmfrfs@...il.com>, Martin Kepplinger <martink@...teo.de>, Purism Kernel Team <kernel@...i.sm>, linux-media@...r.kernel.org, devicetree@...r.kernel.org, imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, Robert Chiras <robert.chiras@....com>, "Guoniu.zhou" <guoniu.zhou@....com> Subject: Re: [PATCH v5 00/13] media: imx8: add camera support Hi Frank, On Mon, Jun 09, 2025 at 01:04:25PM -0400, Frank Li wrote: > On Thu, May 22, 2025 at 01:56:38PM -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@....com> > > Laurent Pinchart: > > Do you have chance to check these patches? Only below patch > media: nxp: imx8-isi: Use devm_clk_bulk_get_all() to fetch clocks > missed your review tag. I've just pinged Rob in the v4 thread. > DT team member still have not any comments about your concern by > use devm_clk_bulk_get_all(). I have quite confidence about this > devm_clk_bulk_get_all(), many drivers already switched to use > devm_clk_bulk_get_all() recently. > > Do you need me rebase to v6.16-rc1? The patches apply cleanly on v6.16-rc1, no need to rebase. I'll wait a few more days for Rob's reply. > > Changes in v5: > > - collect Laurent Pinchart's review tags > > - remove empty endpoint in dts > > > > 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() to 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 | 64 ++++ > > .../boot/dts/freescale/imx8qm-mek-ov5640-csi1.dtso | 64 ++++ > > arch/arm64/boot/dts/freescale/imx8qm-mek.dts | 67 ++++ > > arch/arm64/boot/dts/freescale/imx8qm-ss-img.dtsi | 79 +++++ > > arch/arm64/boot/dts/freescale/imx8qm.dtsi | 5 + > > .../boot/dts/freescale/imx8qxp-mek-ov5640-csi.dtso | 63 ++++ > > arch/arm64/boot/dts/freescale/imx8qxp-mek.dts | 45 +++ > > arch/arm64/boot/dts/freescale/imx8qxp-ss-img.dtsi | 83 +++++ > > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 5 + > > .../media/platform/nxp/imx8-isi/imx8-isi-core.c | 135 ++++---- > > .../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, 1334 insertions(+), 94 deletions(-) > > --- > > base-commit: ae1fdcf4b210cd66ac7e97b24219c160e4bb7be5 > > change-id: 20250114-8qxp_camera-c1af5749d304 -- Regards, Laurent Pinchart
Powered by blists - more mailing lists