From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: Re: [PATCH v6 0/3] Add RTC DT node to Exynos5420 SoC and Fix RTC DT node name for Exynos5250 Date: Mon, 26 Aug 2013 02:23:57 +0900 Message-ID: <521A3DAD.4030301@samsung.com> References: <1377176343-24859-1-git-send-email-vikas.sajjan@linaro.org> <2046167.RNzj0dOWkj@amdc1227> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org To: Vikas Sajjan Cc: Kukjin Kim , Vikas Sajjan , linux-samsung-soc , devicetree@vger.kernel.org, patches@linaro.org, linaro-kernel@lists.linaro.org, Tomasz Figa List-Id: devicetree@vger.kernel.org On 08/23/13 11:49, Vikas Sajjan wrote: > Hi Kukjin, > > On Thu, Aug 22, 2013 at 6:37 PM, Tomasz Figa wrote: >> 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 > > > Can you apply this patchset. > Applied this whole series. Thanks, Kukjin