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. -- 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