Re: [PATCH RESEND v2 0/4] clk/exynos convert clock IDs to macros

[Kukjin Kim <kgene.kim@samsung.com>]

On 02/24/14 22:22, Andrzej Hajda wrote:
> Hi,
>
Hi Andrzej,

> This patch set converts dts files to use macros instead
> of magic numbers or enums to describe clock bindings.
>
> The patch set is rebased on the latest kgene/for-next branch.
>
Thanks.

> The patches are generated by script.
> Many clocks I have verified by hand.
> I have also tested it successfully on exynos4 based board.
>
:)

> This is the 2nd version of the patchset.
> Changes:
> - corrected devicetree mailing list,
> - added comments to include/dt-bindings/clock/exynos4.h for
>    clocks present only in particular chip,
> - added tab alignement in headers,
> - added comment to CLK_NR_CLKS,
> - added copyright headers,
> - split long lines in dts,
> - corrected example in bindings/clock/exynos5250-clock.txt, to point
>    appropriate clocks.
>
> Regards
> Andrzej
>
> Andrzej Hajda (4):
>    ARM: dts: exynos4: convert magic numbers to macros in clock bindings
>    ARM: dts: exynos5250: convert magic numbers to macros in clock
>      bindings
>    ARM: dts: exynos5420: convert magic numbers to macros in clock
>      bindings
>    ARM: dts: exynos5440: convert magic numbers to macros in clock
>      bindings
>
>   .../devicetree/bindings/clock/exynos4-clock.txt    | 259 +--------------------
>   .../devicetree/bindings/clock/exynos5250-clock.txt | 163 +------------
>   .../devicetree/bindings/clock/exynos5420-clock.txt | 184 +--------------
>   .../devicetree/bindings/clock/exynos5440-clock.txt |  45 +---
>   arch/arm/boot/dts/exynos4.dtsi                     |  73 +++---
>   arch/arm/boot/dts/exynos4210.dtsi                  |   9 +-
>   arch/arm/boot/dts/exynos4x12.dtsi                  |  34 +--
>   arch/arm/boot/dts/exynos5250.dtsi                  | 105 +++++----
>   arch/arm/boot/dts/exynos5420.dtsi                  |  95 ++++----
>   arch/arm/boot/dts/exynos5440.dtsi                  |  33 +--
>   10 files changed, 207 insertions(+), 793 deletions(-)
>

Looks good to me, applied.

Thanks,
Kukjin