From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrzej Hajda Subject: Re: [PATCH v8 3/3] arm64: dts: exynos: Add support for S6E3HA2 panel device on TM2 board Date: Wed, 11 Jan 2017 10:39:33 +0100 Message-ID: <581db2a8-e357-a3ba-9464-fa8f83f95c09@samsung.com> References: <1484116439-7275-1-git-send-email-hoegeun.kwon@samsung.com> <1484116439-7275-4-git-send-email-hoegeun.kwon@samsung.com> <5875EF69.1070803@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-reply-to: <5875EF69.1070803@samsung.com> Sender: linux-kernel-owner@vger.kernel.org To: Inki Dae , Hoegeun Kwon , krzk@kernel.org, robh@kernel.org, thierry.reding@gmail.com, airlied@linux.ie, kgene@kernel.org Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, cw00.choi@samsung.com, jh80.chung@samsung.com, andi.shyti@samsung.com, Hyungwon Hwang List-Id: devicetree@vger.kernel.org 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 >>> >>> This patch add the panel device tree node for S6E3HA2 display >>> controller to TM2 dts. >>> >>> Signed-off-by: Hyungwon Hwang >>> Signed-off-by: Andrzej Hajda >>> Signed-off-by: Chanwoo Choi >>> Signed-off-by: Hoegeun Kwon >>> Tested-by: Chanwoo Choi >>> --- >>> 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 > >> Regards >> Andrzej >>> + }; >>> +}; >>> + >>> &hsi2c_9 { >>> status = "okay"; >>> >> >> >> >