Skip to contentSkip to conversation

ARM: dts: imx6qdl-sabreauto: Add HDMI support

PATCH New Fabio Estevam 2 participants 7 messages Jun 2, 2015 – Jul 8, 2015

Subsystems (2)
lore
Share & tools
lore

Fabio Estevam

author reviewer
#0 lore raw

PATCH 1/1 ARM: dts: imx6qdl-sabreauto: Add HDMI support

From: Fabio Estevam <[email protected]>

Add HDMI support. As the DDC I2C pins are not connected due to pin conflicts, we do not pass the 'ddc-i2c-bus' property in this case.

---
 arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/boot/dts/imx6qdl-sabreauto.dtsi +4 -0
index 46b2fed..48ccfee 100644
@@ -76,6 +76,10 @@
 	status = "okay";
 };
 
+&hdmi {
+	status = "okay";
+};
+
 &i2c2 {
 	clock-frequency = <100000>;
 	pinctrl-names = "default";
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 ARM: dts: imx6qdl-sabreauto: Add HDMI support

Quoted 5 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Add HDMI support. As the DDC I2C pins are not connected due to pin conflicts, we do not pass the 'ddc-i2c-bus' property in this case.

HDMI works without 'ddc-i2c-bus' property and I2C pins? If not, what's point of this patch.

Shawn

Quoted 29 lines
---
 arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/boot/dts/imx6qdl-sabreauto.dtsi +4 -0
index 46b2fed..48ccfee 100644
@@ -76,6 +76,10 @@
 	status = "okay";
 };
 
+&hdmi {
+	status = "okay";
+};
+
 &i2c2 {
 	clock-frequency = <100000>;
 	pinctrl-names = "default";
Signature · 7 lines
1.9.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
author reviewer
#2 lore raw

PATCH 1/1 ARM: dts: imx6qdl-sabreauto: Add HDMI support

Quoted 9 lines · Shawn Guo
Quoted 5 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Add HDMI support. As the DDC I2C pins are not connected due to pin conflicts, we do not pass the 'ddc-i2c-bus' property in this case.

HDMI works without 'ddc-i2c-bus' property and I2C pins? If not, what's point of this patch.

Yes, HDMI works fine without 'ddc-i2c-bus' property.

'ddc-i2c-bus' property is not mandatory to get HDMI functional.

Regards,

Fabio Estevam

author reviewer
#3 lore raw

PATCH 1/1 ARM: dts: imx6qdl-sabreauto: Add HDMI support

Quoted 8 lines · Fabio Estevam
Quoted 7 lines · Shawn Guo
Quoted 5 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Add HDMI support. As the DDC I2C pins are not connected due to pin conflicts, we do not pass the 'ddc-i2c-bus' property in this case.

Quoted 6 lines
Quoted 2 lines

HDMI works without 'ddc-i2c-bus' property and I2C pins? If not, what's point of this patch.

Yes, HDMI works fine without 'ddc-i2c-bus' property.

'ddc-i2c-bus' property is not mandatory to get HDMI functional.

You can also refer to Documentation/devicetree/bindings/drm/imx/hdmi.txt, which states that ddc-i2c-bus is an optional property.

#4 lore raw

PATCH 1/1 ARM: dts: imx6qdl-sabreauto: Add HDMI support

In reply to Fabio Estevam
Quoted 8 lines · Fabio Estevam
Quoted 7 lines · Shawn Guo
Quoted 5 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Add HDMI support. As the DDC I2C pins are not connected due to pin conflicts, we do not pass the 'ddc-i2c-bus' property in this case.

Quoted 6 lines
Quoted 2 lines

HDMI works without 'ddc-i2c-bus' property and I2C pins? If not, what's point of this patch.

Yes, HDMI works fine without 'ddc-i2c-bus' property.

'ddc-i2c-bus' property is not mandatory to get HDMI functional.

Okay. Patch applied.

#6 lore raw

PATCH 1/1 ARM: dts: imx6qdl-sabreauto: Add HDMI support

Quoted 10 lines · Fabio Estevam
Quoted 3 lines · Shawn Guo
Quoted 1 line

'ddc-i2c-bus' property is not mandatory to get HDMI functional.

Okay. Patch applied.

Applied into which tree, please?

Can't find it at:
https://git.kernel.org/cgit/linux/kernel/git/shawnguo/linux.git/

I'm rebasing the branches to v4.2-rc1, and will push them out soon.

Shawn