From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1034318AbdAFWLC (ORCPT ); Fri, 6 Jan 2017 17:11:02 -0500 Received: from mail-qk0-f193.google.com ([209.85.220.193]:36304 "EHLO mail-qk0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1034142AbdAFWJx (ORCPT ); Fri, 6 Jan 2017 17:09:53 -0500 MIME-Version: 1.0 Reply-To: cwchoi00@gmail.com In-Reply-To: References: <20170106134350.32428-1-andi.shyti@samsung.com> <20170106134350.32428-6-andi.shyti@samsung.com> From: Chanwoo Choi Date: Sat, 7 Jan 2017 07:09:09 +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-07 6:31 GMT+09:00 Chanwoo Choi : > 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 -- Best Regards, Chanwoo Choi