From patchwork Wed Oct 28 07:37:23 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 7507711 Return-Path: X-Original-To: patchwork-linux-samsung-soc@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 5E0F0BEEA4 for ; Wed, 28 Oct 2015 07:37:35 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 696792081D for ; Wed, 28 Oct 2015 07:37:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 67B532080B for ; Wed, 28 Oct 2015 07:37:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755094AbbJ1Hhc (ORCPT ); Wed, 28 Oct 2015 03:37:32 -0400 Received: from mailout3.w1.samsung.com ([210.118.77.13]:22966 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752109AbbJ1Hhc (ORCPT ); Wed, 28 Oct 2015 03:37:32 -0400 Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout3.w1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0NWX00JCY6IHQF40@mailout3.w1.samsung.com> for linux-samsung-soc@vger.kernel.org; Wed, 28 Oct 2015 07:37:29 +0000 (GMT) X-AuditID: cbfec7f4-f79c56d0000012ee-9c-56307b396b40 Received: from eusync3.samsung.com ( [203.254.199.213]) by eucpsbgm1.samsung.com (EUCPMTA) with SMTP id 83.17.04846.93B70365; Wed, 28 Oct 2015 07:37:29 +0000 (GMT) Received: from [0.0.0.0] ([106.116.37.23]) by eusync3.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id <0NWX0035N6IBIW90@eusync3.samsung.com>; Wed, 28 Oct 2015 07:37:29 +0000 (GMT) Subject: Re: [GIT PULL 4/5] Late Samsung driver updates for v4.4 To: Kukjin Kim , arm@kernel.org, Arnd Bergmann , 'Olof Johansson' , Kevin Hilman References: <562AA40E.4050407@kernel.org> <562AA603.7040600@kernel.org> Cc: "linux-arm-kernel@lists.infradead.org" , "linux-samsung-soc@vger.kernel.org" , p.fedin@samsung.com From: Krzysztof Kozlowski X-Enigmail-Draft-Status: N1110 Message-id: <56307B33.90608@samsung.com> Date: Wed, 28 Oct 2015 16:37:23 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-version: 1.0 In-reply-to: <562AA603.7040600@kernel.org> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrBLMWRmVeSWpSXmKPExsVy+t/xq7qW1QZhBmeXCFsc+7KFzeLvpGPs Fq9fGFr0P37NbPH18ApGi02Pr7FazDi/j8ni1PXPbBavLq1ic+D0+P1rEqPHplWdbB53ru1h 89i8pN7jyokmVo++LasYPT5vkgtgj+KySUnNySxLLdK3S+DK2PP0J0vBecmKwwf7WBoYr4t0 MXJySAiYSKx6e4sNwhaTuHBvPZgtJLCUUeLP7fouRi4g+wujRPvFe8wgCWEBB4nfO18zgyRE BGYxStyZsY6pi5EDqMpdYt92c5A4s8BWRokXN1awgzSwCRhLbF6+BGqDnERv9yQWEJtXQEPi z7Q7rCA2i4CqxLtJs8FqRAUiJCZOaGCFqBGU+DH5Hlg9p4CWRM//n6wgu5gF9CTuX9QCCTML yEtsXvOWeQKj4CwkHbMQqmYhqVrAyLyKUTS1NLmgOCk911CvODG3uDQvXS85P3cTIyQ+vuxg XHzM6hCjAAejEg+vQYVemBBrYllxZe4hRgkOZiUR3hpBgzAh3pTEyqrUovz4otKc1OJDjNIc LErivHN3vQ8REkhPLEnNTk0tSC2CyTJxcEo1MFYcLchN0Xrf8mqpmW698MEJK296rYqy2JV5 /0ZZ+0aRv9KVTR+fpq5WnvDKfpvcFfez2s5dXWtVBdzEdPvvB/28EmG+i6Gz1OADy+5tVsaR Xts1eHvbjrSnP+22XVPwvvyj+Y86Npv3xRmsil7Gr49989d1zGFMfvVBnC14p3vfNOUE8593 lFiKMxINtZiLihMBJqVGD4sCAAA= Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On 24.10.2015 06:26, Kukjin Kim wrote: > The following changes since commit 6ff33f3902c3b1c5d0db6b1e2c70b6d76fba357f: > > Linux 4.3-rc1 (2015-09-12 16:35:56 -0700) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git > tags/samsung-driver > > for you to fetch changes up to c612d5a7785b307df96938d590c3a1edfb624ccf: > > cpufreq: s5pv210: remove superfluous CONFIG_PM ifdefs (2015-10-24 > 04:34:50 +0900) > > ---------------------------------------------------------------- > Samsung driver udpates for v4.4 > > - add support for exynos SROM controller DT based driver > (drivers/soc/samsung/exynos-srom.c) > : Since the exynos SROM registers are used during Suspend to RAM > so the exynos SROM driver handles the S2R and then we can remove > static mapping for SROM registers. > : Pankaj tested boot and S2R functionality on Peach-Pi chromebook > : Pavel tested on SMDK5410 board > : Krzysztof tested on Trats2 (exynos4412) board > - update Maintainer entry accordingly > - remove superfluous CONFIG_PM ifdefs with Viresh's ack > > ---------------------------------------------------------------- > Bartlomiej Zolnierkiewicz (1): > cpufreq: s5pv210: remove superfluous CONFIG_PM ifdefs > > Pankaj Dubey (8): > ARM: EXYNOS: remove unused static mapping of CMU for exynos5 > ARM: EXYNOS: code cleanup in map.h > dt-bindings: add exynos-srom device tree binding > ARM: dts: add SROM device node for exynos4 > ARM: dts: add SROM device node for exynos5 Hi Olof and Kukjin, I see that this pull request was not pulled yet. (Un)Fortunately I spotted one mistake in DTS changes above - the length of mapped region is too small by one word. This does not produce any visible effect but actually is wrong because driver will read from unmapped memory. The question is - do you prefer us to send a following up patch or to fix it by amending the commit? Here is a fix which can be squashed into these two commits: -- >8 -- Subject: [PATCH] ARM: dts: Fix size of mapped SROMC memory region on exynos[45] The SROMC memory region contains 5 registers so the size of mapped memory should be 0x14 instead of 0x10. Signed-off-by: Krzysztof Kozlowski --- arch/arm/boot/dts/exynos4.dtsi | 2 +- arch/arm/boot/dts/exynos5.dtsi | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi index 2f31f773b096..ffda7f5118cd 100644 --- a/arch/arm/boot/dts/exynos4.dtsi +++ b/arch/arm/boot/dts/exynos4.dtsi @@ -78,7 +78,7 @@ sromc@12570000 { compatible = "samsung,exynos-srom"; - reg = <0x12570000 0x10>; + reg = <0x12570000 0x14>; }; mipi_phy: video-phy@10020710 { diff --git a/arch/arm/boot/dts/exynos5.dtsi b/arch/arm/boot/dts/exynos5.dtsi index b5d3437922c5..fb52d16fba33 100644 --- a/arch/arm/boot/dts/exynos5.dtsi +++ b/arch/arm/boot/dts/exynos5.dtsi @@ -32,7 +32,7 @@ sromc@12250000 { compatible = "samsung,exynos-srom"; - reg = <0x12250000 0x10>; + reg = <0x12250000 0x14>; }; combiner: interrupt-controller@10440000 {