All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
@ 2024-04-04  0:20 ` Laurent Pinchart
  0 siblings, 0 replies; 6+ 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] 6+ messages in thread

* [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
@ 2024-04-04  0:20 ` Laurent Pinchart
  0 siblings, 0 replies; 6+ 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


_______________________________________________
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] 6+ messages in thread

* Re: [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
  -1 siblings, 0 replies; 6+ 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] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
@ 2024-04-04  7:42   ` Marco Felsch
  0 siblings, 0 replies; 6+ 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>

_______________________________________________
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] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
  2024-04-04  0:20 ` Laurent Pinchart
@ 2024-04-22  4:03   ` Shawn Guo
  -1 siblings, 0 replies; 6+ 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] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: debix-a: Disable i2c2 in base .dts
@ 2024-04-22  4:03   ` Shawn Guo
  0 siblings, 0 replies; 6+ 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!


_______________________________________________
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] 6+ messages in thread

end of thread, other threads:[~2024-04-22  4:04 UTC | newest]

Thread overview: 6+ 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  0:20 ` Laurent Pinchart
2024-04-04  7:42 ` Marco Felsch
2024-04-04  7:42   ` Marco Felsch
2024-04-22  4:03 ` Shawn Guo
2024-04-22  4:03   ` Shawn Guo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.