On 11.01.2017 11:23, hoegeun kwon wrote: > > On 01/11/2017 06:39 PM, Andrzej Hajda wrote: >> On 11.01.2017 09:40, Inki Dae wrote: >>> 2017년 01월 11일 16:46에 Andrzej Hajda 이(가) 쓴 글: >>>> On 11.01.2017 07:33, Hoegeun Kwon wrote: >>>>> From: Hyungwon Hwang <human.hwang@xxxxxxxxxxx> >>>>> >>>>> This patch add the panel device tree node for S6E3HA2 display >>>>> controller to TM2 dts. >>>>> >>>>> Signed-off-by: Hyungwon Hwang <human.hwang@xxxxxxxxxxx> >>>>> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx> >>>>> Signed-off-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx> >>>>> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@xxxxxxxxxxx> >>>>> Tested-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx> >>>>> --- >>>>> arch/arm64/boot/dts/exynos/exynos5433-tm2.dts | 12 ++++++++++++ >>>>> 1 file changed, 12 insertions(+) >>>>> >>>>> diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts b/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts >>>>> index ddba2f8..6d362f9 100644 >>>>> --- a/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts >>>>> +++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts >>>>> @@ -18,6 +18,18 @@ >>>>> compatible = "samsung,tm2", "samsung,exynos5433"; >>>>> }; >>>>> >>>>> +&dsi { >>>>> + panel@0 { >>>>> + compatible = "samsung,s6e3ha2"; >>>>> + reg = <0>; >>>>> + vdd3-supply = <&ldo27_reg>; >>>>> + vci-supply = <&ldo28_reg>; >>>>> + reset-gpios = <&gpg0 0 GPIO_ACTIVE_LOW>; >>>>> + enable-gpios = <&gpf1 5 GPIO_ACTIVE_HIGH>; >>>>> + te-gpios = <&gpf1 3 GPIO_ACTIVE_HIGH>; >>>> The same here (as in 1st comment) , te-gpios should be dropper - decon >>>> uses hw-trigger. >>> Reasonable to remove te-gpios property but this change would make MIPI-DSI driver probing to be failed so MIPI-DSI driver should be fixed together. >>> >>> Thanks. >> OK, I forgot it was not yet ported to mainline. >> >> Regards >> Andrzej > I received a reply while I was writing the mail. > so, how about removing te-gpios later? I think this is a good solution, just do not forget to change it to optional in bindings, to make it removable. Regards Andrzej -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html