Re: [PATCH v6 0/3] Add RTC DT node to Exynos5420 SoC and Fix RTC DT node name for Exynos5250

[Tomasz Figa <t.figa@samsung.com>]

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 <t.figa@samsung.com>
> 
> 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 <kgene@kernel.org>.
> 
> 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 <t.figa@samsung.com>

Best regards,
Tomasz