Re: [PATCH 3/5] ARM: dts: Add clock entries to G2D node for exynos4210

[Sylwester Nawrocki <s.nawrocki@samsung.com>]

On 04/18/2013 01:54 PM, Sachin Kamat wrote:
>>> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
>>> index 66e6b03..50ab9d4 100644
>>> --- a/arch/arm/boot/dts/exynos4210.dtsi
>>> +++ b/arch/arm/boot/dts/exynos4210.dtsi
>>> @@ -117,6 +117,8 @@
>>>               compatible = "samsung,s5pv210-g2d";
>>>               reg = <0x12800000 0x1000>;
>>>               interrupts = <0 89 0>;
>>> +             clocks = <&clock 177>, <&clock 277>;
>>> +             clock-names = "sclk_fimg2d", "fimg2d";
>>
>> Same clocks are used for exynos4210 and exynos4212/4412. Wouldn't
>> it work if you have added those properties to exynos4.dtsi, instead
>> of having same entries in exynos4210.dtsi and exynos4x12.dtsi  ?
> 
> The clocks are the same. However, base address of register is
> different for 4210 and 4x12.
> That was the reason 2 separate nodes were created. Do you want me to
> create a common node in exynos4.dtsi with all the common properties
> and separate nodes in exynos4210.dtsi and exynos4x12.dtsi with just
> the 'reg' property?

Ah, ok then. I'm fine either way. The compatible strings are also
different so it's probably better to leave exynos4.dtsi out and
have full nodes listed in both exynos4210/4x12.dtsi.

Thanks,
Sylwester