Re: [alsa-devel] [PATCH V4 4/4] ARM: dts: add clock provider information for i2s controllers in Exynos5250

Hi Doug,

On Tue, Jun 4, 2013 at 1:48 AM, Doug Anderson dianders@chromium.org wrote:

Padmavathi,

On Sun, Jun 2, 2013 at 10:19 PM, Padmavathi Venna padma.v@samsung.com wrote:

...

•           clocks = <&clock_audss EXYNOS_I2S_BUS>,
  

•                   <&clock_audss EXYNOS_I2S_BUS>,
  

•                   <&clock_audss EXYNOS_SCLK_I2S>,
  

•                   <&clock_audss EXYNOS_MOUT_AUDSS>,
  

•                   <&clock_audss EXYNOS_MOUT_I2S>;
  

•           clock-names = "iis", "i2s_opclk0", "i2s_opclk1",
  

•                           "mout_audss", "mout_i2s";
  

Are there bindings for these clocks? Would be nice to see a description for what they are supposed to be.

Yes. They have bindings. I will add the description.

I looked up i2s_opclk0 / i2s_opclk1 and they look reasonable at a quick glance. ...and it does seem right that iis and i2s_opclk0 are the same. ...but I don't see any place that uses the last two. Is there an in-flight patch?

I don't have any patch for these right now. These are used internally. But as these are i2s0 base clocks I added here.

-Doug

Thanks Padma