From patchwork Tue Jan 3 15:29:23 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chanwoo Choi X-Patchwork-Id: 9495097 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 93096606B5 for ; Tue, 3 Jan 2017 16:01:50 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 867CF2793B for ; Tue, 3 Jan 2017 16:01:50 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7B0FF279E0; Tue, 3 Jan 2017 16:01:50 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.3 required=2.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_HI, RCVD_IN_SORBS_SPAM, T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1EC5C27AB2 for ; Tue, 3 Jan 2017 16:01:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934517AbdACP7C (ORCPT ); Tue, 3 Jan 2017 10:59:02 -0500 Received: from mail-qk0-f195.google.com ([209.85.220.195]:35658 "EHLO mail-qk0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759097AbdACP54 (ORCPT ); Tue, 3 Jan 2017 10:57:56 -0500 Received: by mail-qk0-f195.google.com with SMTP id u25so50389556qki.2; Tue, 03 Jan 2017 07:57:30 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:reply-to:in-reply-to:references:from:date:message-id :subject:to:cc; bh=nF3jFjE4Q8QDSKoksf3bADhVzV0V9EcohGxZCw0SYes=; b=er+Zp6vhw/Kaj1CU0U2L3pWEkGP3K9ijP5I3dc273mbnA/nPY4hBGtziuX8u0JTQU5 Th2t7fkVraFFy5o9aPuV0zQVnSCirNGHNaVGolovibMgYOdYfgHS1dSEc7Igc2RTfZn7 R8MA9JnjPnsz4WEOuS1mTkmifj7TyFPYKKZ/HNwJCVtoTWSZ02mZt0bacHlkDZL1ChmO gJNTl4H2eZDUUTbEixg8MDazkmn3B7P+Z5JUplB4ZAQ2bb3hCSGxuO5V7ZU+g5BXtAoI dprdxecRzT4sKgeM8FCBq9cl030PnExEExiDfBCnvwPhT9hsPAecWOB/+oVTNfT/3Dzr yY4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:reply-to:in-reply-to:references :from:date:message-id:subject:to:cc; bh=nF3jFjE4Q8QDSKoksf3bADhVzV0V9EcohGxZCw0SYes=; b=YGf7uGO1qYq3BIg4jhqU9soZZq/r5zaKAcs6niRiixJobya4IETR521VjUSABXcgsg 9N83AAvWO1HL131oXDojaiylcefGzjB1Pp/EZ4XlzY8wdWU6283zSGgyW2dN3sSoIiLX AxXi5Fi4/Hyp8fzyIf16qVz29ZkE2wfP47nUtl2xCQ7HMQoLnGXXyVcmsZwa+SHYPaq2 Qwtm5SU6QtG2CIgrzvdu9Lk5d/1NU4O0a/zS7chebXIdUCRoxZCJO2uWEpsB4R+oThx8 BaZmRJ15EehErrBWdc041gLIXC9hsP6qJx5tylHn0f8JZ8JFLOqYZkU/Q5T+aHvUNKTX NGKg== X-Gm-Message-State: AIkVDXI/c6HS3muNKz/OyuNgkpYZPBQ8fWgyN0oulugcRf0s1dnuFerh6N4wIKpwEk/er/ApoyVmBhheDHNbsA== X-Received: by 10.55.44.133 with SMTP id s127mr59854556qkh.131.1483457384324; Tue, 03 Jan 2017 07:29:44 -0800 (PST) MIME-Version: 1.0 Received: by 10.12.129.34 with HTTP; Tue, 3 Jan 2017 07:29:23 -0800 (PST) Reply-To: cwchoi00@gmail.com In-Reply-To: <20170103144056.4ft2ohmhgmezeney@jack.zhora.eu> References: <1483430237-26823-1-git-send-email-jcsing.lee@samsung.com> <1483430237-26823-4-git-send-email-jcsing.lee@samsung.com> <20170103095842.h3hl64amje4qv4ts@gangnam.samsung> <20170103102548.73jg6qddlcthe2mu@gangnam.samsung> <20170103115530.zhjwn7bzmqmumy23@gangnam.samsung> <20170103144056.4ft2ohmhgmezeney@jack.zhora.eu> From: Chanwoo Choi Date: Wed, 4 Jan 2017 00:29:23 +0900 Message-ID: Subject: Re: [PATCH 3/4] arm64: dts: exynos: make tm2 and tm2e independent from each other To: Andi Shyti Cc: Javier Martinez Canillas , Andi Shyti , Mark Rutland , devicetree , linux-samsung-soc , Dmitry Torokhov , Catalin Marinas , Jaechul Lee , Will Deacon , linux-kernel , Krzysztof Kozlowski , Chanwoo Choi , Rob Herring , Kukjin Kim , linux-input@vger.kernel.org, galaxyra@gmail.com, beomho.seo@samsung.com, linux-arm-kernel Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hi Andi, 2017-01-03 23:40 GMT+09:00 Andi Shyti : > Hi, > >> FWIW, I also agree with Chanwoo that the difference is too small to >> need a common .dtsi file. > > in principle I don't like "switching on and off" properties by > overwriting them with "status = disable", unless it's really > necessary (and this case is not). Even for small differences. It > makes the DTS harder to read and duplicates nodes with different > values throughout the DTS include chain. > > In my opinion this approach should be discouraged. > > Besides, there are other overwritten differences in tm2e.dts that > I think should be separated as well. The "common" file approach is > widely used in arm/boot/dts/exynos* files. > > The "status = disable" looks to me more like a temporary hack > rather than a permanent solution. > > In any case, still up to you :) > > Andi I think that "status=disabled" of hsi2c_9 is not hack. The overwrite is possible for Device-tree. But, there is just difference how to support them with some method. Except for touchkey, all peripheral device are same on both tm2 and tm2e. There are only small difference for a few property value. To understand the difference between tm2 and tm2e, I made the patch (it is not complete version). If we implement the following patch, we support both tm2 and tm2e. So, I think that it is not complex to understand the h/w difference between tm2 and tm2e. diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts b/arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts index 1db4e7f..09b6935 100644 --- a/arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts +++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts @@ -18,6 +18,17 @@ compatible = "samsung,tm2e", "samsung,exynos5433"; }; +&display_timings { + clock-frequency = <16523724>; + hactive = <1600>; +}; + +&hsi2c_9 { + /* TM2E don't use the separate touchkey device. Instead, touchscreen + * device support the touchkey device.*/ + status = "disabled"; +}; + &ldo23_reg { regulator-name = "CAM_SEN_CORE_1.025V_AP"; regulator-max-microvolt = <1050000>; @@ -39,3 +50,7 @@ regulator-min-microvolt = <3300000>; regulator-max-microvolt = <3300000>; }; + +&touchscreen { + x-size = "1599"; +};