2023-06-06 15:50:32

by Tim Harvey

[permalink] [raw]
Subject: [PATCH] arm64: dts: imx8mm-venice-gw7904: disable disp_blk_ctrl

The GW7904 does not connect the VDD_MIPI power rails thus MIPI is
disabled. However we must also disable disp_blk_ctrl as it uses the
pgc_mipi power domain and without it being disabled imx8m-blk-ctrl will
fail to probe:
imx8m-blk-ctrl 32e28000.blk-ctrl: error -ETIMEDOUT: failed to attach
power domain "mipi-dsi"
imx8m-blk-ctrl: probe of 32e28000.blk-ctrl failed with error -110

Fixes: b999bdaf0597 ("arm64: dts: imx: Add i.mx8mm Gateworks gw7904 dts support")
Signed-off-by: Tim Harvey <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
index c12e3f4f800f..6ca52a14a5ab 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
@@ -628,6 +628,10 @@ &pcie0 {
status = "okay";
};

+&disp_blk_ctrl {
+ status = "disabled";
+};
+
&pgc_mipi {
status = "disabled";
};
--
2.25.1



2023-07-17 01:04:28

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mm-venice-gw7904: disable disp_blk_ctrl

On Tue, Jun 06, 2023 at 08:40:30AM -0700, Tim Harvey wrote:
> The GW7904 does not connect the VDD_MIPI power rails thus MIPI is
> disabled. However we must also disable disp_blk_ctrl as it uses the
> pgc_mipi power domain and without it being disabled imx8m-blk-ctrl will
> fail to probe:
> imx8m-blk-ctrl 32e28000.blk-ctrl: error -ETIMEDOUT: failed to attach
> power domain "mipi-dsi"
> imx8m-blk-ctrl: probe of 32e28000.blk-ctrl failed with error -110
>
> Fixes: b999bdaf0597 ("arm64: dts: imx: Add i.mx8mm Gateworks gw7904 dts support")
> Signed-off-by: Tim Harvey <[email protected]>

Applied, thanks!