From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH 5/5] ARM: dts: Add dts files for Exynos4415 SoC Date: Fri, 24 Oct 2014 15:30:01 +0200 Message-ID: <544A5459.9020006@gmail.com> References: <1413775749-17539-1-git-send-email-cw00.choi@samsung.com> <1413775935-17743-5-git-send-email-cw00.choi@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f170.google.com ([209.85.212.170]:62029 "EHLO mail-wi0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756444AbaJXNaG (ORCPT ); Fri, 24 Oct 2014 09:30:06 -0400 In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Daniel Drake , Chanwoo Choi Cc: linux-samsung-soc , Linux Kernel , "linux-arm-kernel@lists.infradead.org" , Kukjin Kim , ben-linux@fluff.org, Russell King , Mark Rutland , Arnd Bergmann , Olof Johansson , Mike Turquette , thomas.abraham@linaro.org, Linus Walleij , Seung-Woo Kim , Kyungmin Park , InKi Dae , geunsik.lim@samsung.com, jh80.chung@samsung.com, jaewon02.kim@samsung.com, ideal.song@samsung.com, yj44.cho@samsung.com, Marek Szyprowski On 24.10.2014 15:23, Daniel Drake wrote: > On Sun, Oct 19, 2014 at 9:32 PM, Chanwoo Choi wrote: >> This patch adds new exynos4415.dtsi to support Exynos4415 SoC >> based on Cortex-A9 quad cores and includes following dt nodes: > > There's a lot in common between your new exynos4415.dtsi and the > existing exynos4.dtsi. > Would it make more sense for the 4415 code to extend the existing > exynos4.dtsi like the other Exynos4 variants do? This would make sense, but then existing Exynos 4 device tree sources would have to be refactored to use reference-based syntax for extending nodes. (Which is desirable anyway, but adds quite a bit of effort and could be prone to conflicts.) I'd say that they could be merged later anyway. Best regards, Tomasz From mboxrd@z Thu Jan 1 00:00:00 1970 From: tomasz.figa@gmail.com (Tomasz Figa) Date: Fri, 24 Oct 2014 15:30:01 +0200 Subject: [PATCH 5/5] ARM: dts: Add dts files for Exynos4415 SoC In-Reply-To: References: <1413775749-17539-1-git-send-email-cw00.choi@samsung.com> <1413775935-17743-5-git-send-email-cw00.choi@samsung.com> Message-ID: <544A5459.9020006@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 24.10.2014 15:23, Daniel Drake wrote: > On Sun, Oct 19, 2014 at 9:32 PM, Chanwoo Choi wrote: >> This patch adds new exynos4415.dtsi to support Exynos4415 SoC >> based on Cortex-A9 quad cores and includes following dt nodes: > > There's a lot in common between your new exynos4415.dtsi and the > existing exynos4.dtsi. > Would it make more sense for the 4415 code to extend the existing > exynos4.dtsi like the other Exynos4 variants do? This would make sense, but then existing Exynos 4 device tree sources would have to be refactored to use reference-based syntax for extending nodes. (Which is desirable anyway, but adds quite a bit of effort and could be prone to conflicts.) I'd say that they could be merged later anyway. Best regards, Tomasz