From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sylwester Nawrocki Subject: Re: [PATCH 3/5] ARM: dts: Add clock entries to G2D node for exynos4210 Date: Thu, 18 Apr 2013 15:27:47 +0200 Message-ID: <516FF4D3.8030906@samsung.com> References: <1366283719-9615-1-git-send-email-sachin.kamat@linaro.org> <1366283719-9615-3-git-send-email-sachin.kamat@linaro.org> <516FDB1D.3050302@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:40957 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755554Ab3DRN1u (ORCPT ); Thu, 18 Apr 2013 09:27:50 -0400 Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout3.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MLG00BN9DD5AD00@mailout3.w1.samsung.com> for linux-samsung-soc@vger.kernel.org; Thu, 18 Apr 2013 14:27:48 +0100 (BST) In-reply-to: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Sachin Kamat Cc: linux-samsung-soc@vger.kernel.org, kgene.kim@samsung.com, patches@linaro.org On 04/18/2013 01:54 PM, Sachin Kamat wrote: >>> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi >>> index 66e6b03..50ab9d4 100644 >>> --- a/arch/arm/boot/dts/exynos4210.dtsi >>> +++ b/arch/arm/boot/dts/exynos4210.dtsi >>> @@ -117,6 +117,8 @@ >>> compatible = "samsung,s5pv210-g2d"; >>> reg = <0x12800000 0x1000>; >>> interrupts = <0 89 0>; >>> + clocks = <&clock 177>, <&clock 277>; >>> + clock-names = "sclk_fimg2d", "fimg2d"; >> >> Same clocks are used for exynos4210 and exynos4212/4412. Wouldn't >> it work if you have added those properties to exynos4.dtsi, instead >> of having same entries in exynos4210.dtsi and exynos4x12.dtsi ? > > The clocks are the same. However, base address of register is > different for 4210 and 4x12. > That was the reason 2 separate nodes were created. Do you want me to > create a common node in exynos4.dtsi with all the common properties > and separate nodes in exynos4210.dtsi and exynos4x12.dtsi with just > the 'reg' property? Ah, ok then. I'm fine either way. The compatible strings are also different so it's probably better to leave exynos4.dtsi out and have full nodes listed in both exynos4210/4x12.dtsi. Thanks, Sylwester