From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933919AbdAFVbx (ORCPT ); Fri, 6 Jan 2017 16:31:53 -0500 Received: from mail-qk0-f196.google.com ([209.85.220.196]:36667 "EHLO mail-qk0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750779AbdAFVbn (ORCPT ); Fri, 6 Jan 2017 16:31:43 -0500 MIME-Version: 1.0 Reply-To: cwchoi00@gmail.com In-Reply-To: <20170106134350.32428-6-andi.shyti@samsung.com> References: <20170106134350.32428-1-andi.shyti@samsung.com> <20170106134350.32428-6-andi.shyti@samsung.com> From: Chanwoo Choi Date: Sat, 7 Jan 2017 06:31:21 +0900 Message-ID: Subject: Re: [PATCH v5 5/5] arm64: dts: exynos: Add tm2 touchkey node To: Andi Shyti Cc: Dmitry Torokhov , Rob Herring , Mark Rutland , Catalin Marinas , Will Deacon , Kukjin Kim , Krzysztof Kozlowski , Javier Martinez Canillas , Chanwoo Choi , Beomho Seo , linux-arm-kernel , linux-input@vger.kernel.org, devicetree , linux-kernel , linux-samsung-soc , Jaechul Lee , Andi Shyti , Jaechul Lee Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, 2017-01-06 22:43 GMT+09:00 Andi Shyti : > From: Jaechul Lee > > Add DT node support for TM2 touchkey device. > > Signed-off-by: Beomho Seo > Signed-off-by: Jaechul Lee > Signed-off-by: Andi Shyti > Reviewed-by: Javier Martinez Canillas > --- > arch/arm64/boot/dts/exynos/exynos5433-tm2.dts | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts b/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts > index 2449266b268f..92fcc4ec8319 100644 > --- a/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts > +++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts > @@ -18,6 +18,19 @@ > compatible = "samsung,tm2", "samsung,exynos5433"; > }; > > +&hsi2c_9 { > + status = "okay"; > + > + touchkey@20 { > + compatible = "samsung,tm2-touchkey"; > + reg = <0x20>; > + interrupt-parent = <&gpa3>; > + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > + vcc-supply = <&ldo32_reg>; > + vdd-supply = <&ldo33_reg>; > + }; > +}; > + > &ldo31_reg { > regulator-name = "TSP_VDD_1.85V_AP"; > regulator-min-microvolt = <1850000>; Looks good to me. Reviewed-by: Chanwoo Choi -- Best Regards, Chanwoo Choi