Commit 0ff08803 authored by Liu Ying's avatar Liu Ying Committed by Shawn Guo

arm64: dts: imx8mp-evk: Fix hdmi@3d node

The hdmi@3d node's compatible string is "adi,adv7535" instead of
"adi,adv7533" or "adi,adv751*".

Fix the hdmi@3d node by means of:
* Use default register addresses for "cec", "edid" and "packet", because
  there is no need to use a non-default address map.
* Add missing interrupt related properties.
* Drop "adi,input-*" properties which are only valid for adv751*.
* Add VEXT_3V3 fixed regulator.
* Add "*-supply" properties, since most are required.
* Fix label names - s/adv7533/adv7535/.

Fixes: 65344b9b ("arm64: dts: imx8mp-evk: Add HDMI support")
Signed-off-by: default avatarLiu Ying <victor.liu@nxp.com>
Signed-off-by: default avatarShawn Guo <shawnguo@kernel.org>
parent a05de2cc
...@@ -23,7 +23,7 @@ hdmi-connector { ...@@ -23,7 +23,7 @@ hdmi-connector {
port { port {
hdmi_connector_in: endpoint { hdmi_connector_in: endpoint {
remote-endpoint = <&adv7533_out>; remote-endpoint = <&adv7535_out>;
}; };
}; };
}; };
...@@ -107,6 +107,13 @@ reg_usdhc2_vmmc: regulator-usdhc2 { ...@@ -107,6 +107,13 @@ reg_usdhc2_vmmc: regulator-usdhc2 {
enable-active-high; enable-active-high;
}; };
reg_vext_3v3: regulator-vext-3v3 {
compatible = "regulator-fixed";
regulator-name = "VEXT_3V3";
regulator-min-microvolt = <3300000>;
regulator-max-microvolt = <3300000>;
};
sound { sound {
compatible = "simple-audio-card"; compatible = "simple-audio-card";
simple-audio-card,name = "wm8960-audio"; simple-audio-card,name = "wm8960-audio";
...@@ -364,7 +371,7 @@ BUCK4 { ...@@ -364,7 +371,7 @@ BUCK4 {
regulator-always-on; regulator-always-on;
}; };
BUCK5 { reg_buck5: BUCK5 {
regulator-name = "BUCK5"; regulator-name = "BUCK5";
regulator-min-microvolt = <1650000>; regulator-min-microvolt = <1650000>;
regulator-max-microvolt = <1950000>; regulator-max-microvolt = <1950000>;
...@@ -415,14 +422,16 @@ &i2c2 { ...@@ -415,14 +422,16 @@ &i2c2 {
hdmi@3d { hdmi@3d {
compatible = "adi,adv7535"; compatible = "adi,adv7535";
reg = <0x3d>, <0x3c>, <0x3e>, <0x3f>; reg = <0x3d>;
reg-names = "main", "cec", "edid", "packet"; interrupt-parent = <&gpio1>;
interrupts = <9 IRQ_TYPE_EDGE_FALLING>;
adi,dsi-lanes = <4>; adi,dsi-lanes = <4>;
adi,input-depth = <8>; avdd-supply = <&reg_buck5>;
adi,input-colorspace = "rgb"; dvdd-supply = <&reg_buck5>;
adi,input-clock = "1x"; pvdd-supply = <&reg_buck5>;
adi,input-style = <1>; a2vdd-supply = <&reg_buck5>;
adi,input-justification = "evenly"; v3p3-supply = <&reg_vext_3v3>;
v1p2-supply = <&reg_buck5>;
ports { ports {
#address-cells = <1>; #address-cells = <1>;
...@@ -431,7 +440,7 @@ ports { ...@@ -431,7 +440,7 @@ ports {
port@0 { port@0 {
reg = <0>; reg = <0>;
adv7533_in: endpoint { adv7535_in: endpoint {
remote-endpoint = <&dsi_out>; remote-endpoint = <&dsi_out>;
}; };
}; };
...@@ -439,7 +448,7 @@ adv7533_in: endpoint { ...@@ -439,7 +448,7 @@ adv7533_in: endpoint {
port@1 { port@1 {
reg = <1>; reg = <1>;
adv7533_out: endpoint { adv7535_out: endpoint {
remote-endpoint = <&hdmi_connector_in>; remote-endpoint = <&hdmi_connector_in>;
}; };
}; };
...@@ -524,7 +533,7 @@ port@1 { ...@@ -524,7 +533,7 @@ port@1 {
reg = <1>; reg = <1>;
dsi_out: endpoint { dsi_out: endpoint {
remote-endpoint = <&adv7533_in>; remote-endpoint = <&adv7535_in>;
data-lanes = <1 2 3 4>; data-lanes = <1 2 3 4>;
}; };
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment