linux-kernel - Re: [PATCH v1] arm64: dts: imx8-ss-img: Avoid gpio0_mipi

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <aO5p3hSnnOv+evtk@lizhi-Precision-Tower-5810>
Date: Tue, 14 Oct 2025 11:18:54 -0400
From: Frank Li <Frank.li@....com>
To: João Paulo Gonçalves <jpaulo.silvagoncalves@...il.com>
Cc: 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>, devicetree@...r.kernel.org,
	imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
	linux-kernel@...r.kernel.org,
	João Paulo Gonçalves <joao.goncalves@...adex.com>
Subject: Re: [PATCH v1] arm64: dts: imx8-ss-img: Avoid gpio0_mipi_csi GPIOs
 being deferred

On Tue, Oct 14, 2025 at 09:56:43AM -0300, João Paulo Gonçalves wrote:
> From: João Paulo Gonçalves <joao.goncalves@...adex.com>
>
> The gpio0_mipi_csi DT nodes are enabled by default, but they are
> dependent on the irqsteer_csi nodes, which are not enabled. This causes
> the gpio0_mipi_csi GPIOs to be probe deferred. Since these GPIOs can be
> used independently of the CSI controller, enable irqsteer_csi by default
> too to prevent them from being deferred and to ensure they work out of
> the box.

If enable irqsteer_csi[0,1] default, all &irqsteer_csi in board file can
be cleanup.

Reviewed-by: Frank Li <Frank.Li@....com>

>
> Fixes: 2217f8243714 ("arm64: dts: imx8: add capture controller for i.MX8's img subsystem")
> Signed-off-by: João Paulo Gonçalves <joao.goncalves@...adex.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
> index 2cf0f7208350a416d77b11140279d2f66f41498f..a72b2f1c4a1b2ef26196c810b33ecfdebdbc1675 100644
> --- a/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
> @@ -67,7 +67,6 @@ irqsteer_csi0: irqsteer@...20000 {
>  		power-domains = <&pd IMX_SC_R_CSI_0>;
>  		fsl,channel = <0>;
>  		fsl,num-irqs = <32>;
> -		status = "disabled";
>  	};
>
>  	gpio0_mipi_csi0: gpio@...22000 {
> @@ -144,7 +143,6 @@ irqsteer_csi1: irqsteer@...40000 {
>  		power-domains = <&pd IMX_SC_R_CSI_1>;
>  		fsl,channel = <0>;
>  		fsl,num-irqs = <32>;
> -		status = "disabled";
>  	};
>
>  	gpio0_mipi_csi1: gpio@...42000 {
>
> ---
> base-commit: 3a8660878839faadb4f1a6dd72c3179c1df56787
> change-id: 20251009-imx8-gpio-csi-deferred-probe-eef6c56a35aa
>
> Best regards,
> --
> João Paulo Gonçalves <joao.goncalves@...adex.com>
>