hdmirx: pinmux later so as to prevent abnormal hpd pulse [1/1]

PD#SWPL-15003

Problem:
There's a pull high pulse of hpd when bootup. Reason:
pinctl-names is set "default" in dts, when bootup,
hdmirx related pinmux will be set by device driver
which is before hdmirx probe. As hpd pin of ic is
default low, after reverse by triode, it will output
high on hdmi receptacle. And after hdmirx probe, hpd
invertion will be enabled before finally output from
ic, so it will become normally low again on hdmi
receptacle. So there's a pull high pulse(1~4ms) output.

Solution:
GPIOW is high(1.4V) before pinmux, so hpd pin of receptacle
is low before pinmux. Need to enable hpd invertion before
GPIOW is muxed to hpd function. the way to delay pinmux
is to change pinctrl-names of dts from default to others.

Verify:
TL1

Change-Id: Ica54484fa4c3da46e7b1ddc72a3c5f44fd4a71f6
Signed-off-by: Hang Cheng <hang.cheng@amlogic.com>
This commit is contained in:
Hang Cheng
2019-10-10 16:49:23 +08:00
committed by Tao Zeng
parent 4e60c444f8
commit 952ac4c1d7
24 changed files with 24 additions and 24 deletions

View File

@@ -534,7 +534,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -830,7 +830,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -820,7 +820,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -876,7 +876,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -870,7 +870,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -871,7 +871,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -865,7 +865,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -570,7 +570,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -868,7 +868,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -872,7 +872,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -850,7 +850,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -855,7 +855,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -856,7 +856,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -818,7 +818,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -873,7 +873,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -866,7 +866,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -867,7 +867,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -860,7 +860,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -570,7 +570,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -864,7 +864,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -872,7 +872,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -848,7 +848,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -855,7 +855,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;

View File

@@ -859,7 +859,7 @@
#size-cells=<1>;
memory-region = <&hdmirx_emp_cma_reserved>;
status = "okay";
pinctrl-names = "default";
pinctrl-names = "hdmirx_pins";
pinctrl-0 = <&hdmirx_a_mux &hdmirx_b_mux
&hdmirx_c_mux>;
repeat = <0>;