On 06/09/2014 03:23 PM, Sylwester Nawrocki wrote:
> Hi Tushar,
>
> On 07/06/14 11:30, Tushar Behera wrote:
>> I2S driver uses 3 clocks under different conditions. Added two
>> missing clocks.
>>
>> Additionally updated pin-control property for this node.
>>
>> Signed-off-by: Tushar Behera <tushar.b@xxxxxxxxxxx>
>> ---
>> Based on next-20140606.
>>
>> Tested on Exynos4210-based Origen boards with private patches for
>> codec/machine driver.
>>
>> arch/arm/boot/dts/exynos4.dtsi | 8 ++++++--
>> 1 file changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
>> index fbaf426..10bb081 100644
>> --- a/arch/arm/boot/dts/exynos4.dtsi
>> +++ b/arch/arm/boot/dts/exynos4.dtsi
>> @@ -55,11 +55,15 @@
>> i2s0: i2s@03830000 {
>> compatible = "samsung,s5pv210-i2s";
>> reg = <0x03830000 0x100>;
>> - clocks = <&clock_audss EXYNOS_I2S_BUS>;
>> - clock-names = "iis";
>> + clocks = <&clock_audss EXYNOS_I2S_BUS>,
>> + <&clock_audss EXYNOS_I2S_BUS>,
>> + <&clock_audss EXYNOS_SCLK_I2S>;
>> + clock-names = "iis", "i2s_opclk0", "i2s_opclk1";
>> dmas = <&pdma0 12>, <&pdma0 11>, <&pdma0 10>;
>> dma-names = "tx", "rx", "tx-sec";
>> samsung,idma-addr = <0x03000000>;
>> + pinctrl-0 = <&i2s0_bus>;
>> + pinctrl-names = "default";
>> status = "disabled";
>> };
>
> I assumed those properties you're adding here belong to board or
> to SoC type specific dts. It doesn't hurt ot have it like this,
> but what do you think about moving this to arch/arm/boot/dts/
> exynos4210(4x12).dtsi ? For instance pinctrl definitions are
> in SoC specific dts.
>
IMHO, the clock entries belong here as the audss clock provider
essentially remains same for all Exynos4 based systems.
As for pin-control entry, I believe we can move them to respective SoC
dtsi files.
Does that sound okay to you?
> --
> Thanks,
> Sylwester
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Tushar Behera
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
