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

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

[Kukjin Kim]

On 08/23/13 11:49, Vikas Sajjan wrote:
> Hi Kukjin,
>
> On Thu, Aug 22, 2013 at 6:37 PM, Tomasz Figa<t.figa@samsung.com>  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<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>
>
>
> Can you apply this patchset.
>

Applied this whole series.

Thanks,
Kukjin

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

[Vikas Sajjan]

Hi Kukjin,

On 25 August 2013 22:53, Kukjin Kim <kgene.kim@samsung.com> wrote:
> On 08/23/13 11:49, Vikas Sajjan wrote:
>>
>> Hi Kukjin,
>>
>> On Thu, Aug 22, 2013 at 6:37 PM, Tomasz Figa<t.figa@samsung.com>  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<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>
>>
>>
>>
>> Can you apply this patchset.
>>
>
> Applied this whole series.

Thanks.


>
> Thanks,
> Kukjin



-- 
Thanks and Regards
 Vikas Sajjan