From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH v6 0/3] Add RTC DT node to Exynos5420 SoC and Fix RTC DT node name for Exynos5250 Date: Thu, 22 Aug 2013 15:07:58 +0200 Message-ID: <2046167.RNzj0dOWkj@amdc1227> References: <1377176343-24859-1-git-send-email-vikas.sajjan@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:55182 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752807Ab3HVNID (ORCPT ); Thu, 22 Aug 2013 09:08:03 -0400 In-reply-to: <1377176343-24859-1-git-send-email-vikas.sajjan@linaro.org> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Vikas Sajjan Cc: linux-samsung-soc@vger.kernel.org, kgene.kim@samsung.com, devicetree@vger.kernel.org, patches@linaro.org, linaro-kernel@lists.linaro.org Hi Vikas, On Thursday 22 of August 2013 18:29:00 Vikas Sajjan wrote: > This patchset does the following 3 things: > 1) Fixes the RTC DT node name for Exynos5250 > 2) Update the "status" property of RTC DT node for Exynos5250 SoC > 3) Adds RTC DT node to Exynos5420 SoC > > changes since v5: > - Fixed the RTC DT node name for Exynos5250 board files also, > because otherwise it breaks RTC support in bisections. > > changes since v4: > - removed "status" property of RTC DT node from exynos5250 board dts > files > > changes since v3: > - split the 5250 related modifications into 2 separate patch as > suggested by Tomasz Figa > > changes since v2: > - split the 5250 related modifications into a separate patch. > - placed the RTC node as per the alphabetical order in the DTS file as > suggested by Kukjin Kim . > > changes since v1: > - made DT node status as "okay" in the dtsi file itself. > > Vikas Sajjan (3): > ARM: dts: Fix the RTC DT node name for Exynos5250 > ARM: dts: Update the "status" property of RTC DT node for Exynos5250 > SoC > ARM: dts: Add RTC DT node to Exynos5420 SoC > > arch/arm/boot/dts/exynos5.dtsi | 2 +- > arch/arm/boot/dts/exynos5250-arndale.dts | 4 ---- > arch/arm/boot/dts/exynos5250-snow.dts | 4 ---- > arch/arm/boot/dts/exynos5250.dtsi | 3 ++- > arch/arm/boot/dts/exynos5420.dtsi | 6 ++++++ > 5 files changed, 9 insertions(+), 10 deletions(-) Looks good to me now. Reviewed-by: Tomasz Figa Best regards, Tomasz