* [PATCH v1] ARM: dts: imx7: remove DSI port endpoints
@ 2024-02-16 10:42 Francesco Dolcini
2024-02-23 17:19 ` Francesco Dolcini
0 siblings, 1 reply; 3+ messages in thread
From: Francesco Dolcini @ 2024-02-16 10:42 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team
Cc: Francesco Dolcini, devicetree, linux-arm-kernel, linux-kernel,
Marco Felsch, Roland Hieber, Hiago De Franco
From: Francesco Dolcini <francesco.dolcini@toradex.com>
This fixes the display not working on colibri imx7, the driver fails to
load with the following error:
mxsfb 30730000.lcdif: error -ENODEV: Cannot connect bridge
NXP i.MX7 LCDIF is connected to both the Parallel LCD Display and to a
MIPI DSI IP block, currently it's not possible to describe the
connection to both.
Remove the port endpoint from the SOC dtsi to prevent regressions, this
would need to be defined on the board DTS.
Reported-by: Hiago De Franco <hiagofranco@gmail.com>
Closes: https://lore.kernel.org/r/34yzygh3mbwpqr2re7nxmhyxy3s7qmqy4vhxvoyxnoguktriur@z66m7gvpqlia/
Fixes: edbbae7fba49 ("ARM: dts: imx7: add MIPI-DSI support")
Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
---
arch/arm/boot/dts/nxp/imx/imx7s.dtsi | 26 --------------------------
1 file changed, 26 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx7s.dtsi b/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
index ebf7befcc11e..9c81c6baa2d3 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
@@ -834,16 +834,6 @@ lcdif: lcdif@30730000 {
<&clks IMX7D_LCDIF_PIXEL_ROOT_CLK>;
clock-names = "pix", "axi";
status = "disabled";
-
- port {
- #address-cells = <1>;
- #size-cells = <0>;
-
- lcdif_out_mipi_dsi: endpoint@0 {
- reg = <0>;
- remote-endpoint = <&mipi_dsi_in_lcdif>;
- };
- };
};
mipi_csi: mipi-csi@30750000 {
@@ -895,22 +885,6 @@ mipi_dsi: dsi@30760000 {
samsung,esc-clock-frequency = <20000000>;
samsung,pll-clock-frequency = <24000000>;
status = "disabled";
-
- ports {
- #address-cells = <1>;
- #size-cells = <0>;
-
- port@0 {
- reg = <0>;
- #address-cells = <1>;
- #size-cells = <0>;
-
- mipi_dsi_in_lcdif: endpoint@0 {
- reg = <0>;
- remote-endpoint = <&lcdif_out_mipi_dsi>;
- };
- };
- };
};
};
--
2.39.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v1] ARM: dts: imx7: remove DSI port endpoints
2024-02-16 10:42 [PATCH v1] ARM: dts: imx7: remove DSI port endpoints Francesco Dolcini
@ 2024-02-23 17:19 ` Francesco Dolcini
2024-02-25 3:38 ` Shawn Guo
0 siblings, 1 reply; 3+ messages in thread
From: Francesco Dolcini @ 2024-02-23 17:19 UTC (permalink / raw)
To: Shawn Guo
Cc: Francesco Dolcini, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Francesco Dolcini, devicetree, linux-arm-kernel,
linux-kernel, Marco Felsch, Roland Hieber, Hiago De Franco,
Arnd Bergmann
On Fri, Feb 16, 2024 at 11:42:55AM +0100, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
>
> This fixes the display not working on colibri imx7, the driver fails to
> load with the following error:
>
> mxsfb 30730000.lcdif: error -ENODEV: Cannot connect bridge
>
> NXP i.MX7 LCDIF is connected to both the Parallel LCD Display and to a
> MIPI DSI IP block, currently it's not possible to describe the
> connection to both.
>
> Remove the port endpoint from the SOC dtsi to prevent regressions, this
> would need to be defined on the board DTS.
>
> Reported-by: Hiago De Franco <hiagofranco@gmail.com>
> Closes: https://lore.kernel.org/r/34yzygh3mbwpqr2re7nxmhyxy3s7qmqy4vhxvoyxnoguktriur@z66m7gvpqlia/
> Fixes: edbbae7fba49 ("ARM: dts: imx7: add MIPI-DSI support")
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
Hello Shawn, what do you plan to do with this?
This fixes a regression from v6.8-rc1 and would be nice to have it into
the final v6.8.
Francesco
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v1] ARM: dts: imx7: remove DSI port endpoints
2024-02-23 17:19 ` Francesco Dolcini
@ 2024-02-25 3:38 ` Shawn Guo
0 siblings, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2024-02-25 3:38 UTC (permalink / raw)
To: Francesco Dolcini
Cc: Shawn Guo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Francesco Dolcini, devicetree, linux-arm-kernel,
linux-kernel, Marco Felsch, Roland Hieber, Hiago De Franco,
Arnd Bergmann
On Fri, Feb 23, 2024 at 06:19:01PM +0100, Francesco Dolcini wrote:
> On Fri, Feb 16, 2024 at 11:42:55AM +0100, Francesco Dolcini wrote:
> > From: Francesco Dolcini <francesco.dolcini@toradex.com>
> >
> > This fixes the display not working on colibri imx7, the driver fails to
> > load with the following error:
> >
> > mxsfb 30730000.lcdif: error -ENODEV: Cannot connect bridge
> >
> > NXP i.MX7 LCDIF is connected to both the Parallel LCD Display and to a
> > MIPI DSI IP block, currently it's not possible to describe the
> > connection to both.
> >
> > Remove the port endpoint from the SOC dtsi to prevent regressions, this
> > would need to be defined on the board DTS.
> >
> > Reported-by: Hiago De Franco <hiagofranco@gmail.com>
> > Closes: https://lore.kernel.org/r/34yzygh3mbwpqr2re7nxmhyxy3s7qmqy4vhxvoyxnoguktriur@z66m7gvpqlia/
> > Fixes: edbbae7fba49 ("ARM: dts: imx7: add MIPI-DSI support")
> > Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
>
> Hello Shawn, what do you plan to do with this?
>
> This fixes a regression from v6.8-rc1 and would be nice to have it into
> the final v6.8.
Applied as a fix for 6.8, thanks!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-02-25 3:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-16 10:42 [PATCH v1] ARM: dts: imx7: remove DSI port endpoints Francesco Dolcini
2024-02-23 17:19 ` Francesco Dolcini
2024-02-25 3:38 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).