* [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
@ 2024-04-04 0:20 Laurent Pinchart
2024-04-04 7:42 ` Marco Felsch
2024-04-22 4:03 ` Shawn Guo
0 siblings, 2 replies; 3+ messages in thread
From: Laurent Pinchart @ 2024-04-04 0:20 UTC (permalink / raw)
To: devicetree, imx, linux-arm-kernel
Cc: Jacopo Mondi, Rob Herring, Conor Dooley, Krzysztof Kozlowski,
Fabio Estevam, Sascha Hauer, Pengutronix Kernel Team,
Jacopo Mondi, Shawn Guo
From: Jacopo Mondi <jacopo@jmondi.org>
The I2C2 bus is used for the CSI and DSI connectors only, no devices are
connected to it on neither the Debix Model A nor its IO board. Disable
the bus in the board's .dts and remove its clock frequency settings, as
the value depends solely on the devices conncted to the CSI and DSI
connectors. Display panel or camera sensor overlays will configure and
enable the bus when necessary.
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
Changes since v1:
- Don't drop the bus, just disable it
---
arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts b/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts
index 2c19766ebf09..9b8f97a84e61 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-debix-model-a.dts
@@ -197,10 +197,8 @@ ldo5: LDO5 {
};
&i2c2 {
- clock-frequency = <100000>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_i2c2>;
- status = "okay";
};
&i2c3 {
base-commit: 4cece764965020c22cff7665b18a012006359095
--
Regards,
Laurent Pinchart
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
2024-04-04 0:20 [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts Laurent Pinchart
@ 2024-04-04 7:42 ` Marco Felsch
2024-04-22 4:03 ` Shawn Guo
1 sibling, 0 replies; 3+ messages in thread
From: Marco Felsch @ 2024-04-04 7:42 UTC (permalink / raw)
To: Laurent Pinchart
Cc: devicetree, imx, linux-arm-kernel, Rob Herring, Jacopo Mondi,
Pengutronix Kernel Team, Shawn Guo, Sascha Hauer, Conor Dooley,
Krzysztof Kozlowski, Jacopo Mondi, Fabio Estevam
On 24-04-04, Laurent Pinchart wrote:
> From: Jacopo Mondi <jacopo@jmondi.org>
>
> The I2C2 bus is used for the CSI and DSI connectors only, no devices are
> connected to it on neither the Debix Model A nor its IO board. Disable
> the bus in the board's .dts and remove its clock frequency settings, as
> the value depends solely on the devices conncted to the CSI and DSI
> connectors. Display panel or camera sensor overlays will configure and
> enable the bus when necessary.
>
> Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
2024-04-04 0:20 [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts Laurent Pinchart
2024-04-04 7:42 ` Marco Felsch
@ 2024-04-22 4:03 ` Shawn Guo
1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2024-04-22 4:03 UTC (permalink / raw)
To: Laurent Pinchart
Cc: devicetree, imx, linux-arm-kernel, Jacopo Mondi, Rob Herring,
Conor Dooley, Krzysztof Kozlowski, Fabio Estevam, Sascha Hauer,
Pengutronix Kernel Team, Jacopo Mondi, Shawn Guo
On Thu, Apr 04, 2024 at 03:20:09AM +0300, Laurent Pinchart wrote:
> From: Jacopo Mondi <jacopo@jmondi.org>
>
> The I2C2 bus is used for the CSI and DSI connectors only, no devices are
> connected to it on neither the Debix Model A nor its IO board. Disable
> the bus in the board's .dts and remove its clock frequency settings, as
> the value depends solely on the devices conncted to the CSI and DSI
> connectors. Display panel or camera sensor overlays will configure and
> enable the bus when necessary.
>
> Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Applied, thanks!
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-04-22 4:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-04 0:20 [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts Laurent Pinchart
2024-04-04 7:42 ` Marco Felsch
2024-04-22 4:03 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox