Hi Doug,

On Tue, Jun 4, 2013 at 10:49 PM, Doug Anderson dianders@chromium.org wrote:

Padmavathi,

On Tue, Jun 4, 2013 at 5:28 AM, Padmavathi Venna padma.v@samsung.com wrote:

...

@@ -471,6 +477,8 @@ dmas = <&pdma1 12 &pdma1 11>; dma-names = "tx", "rx";

•           clocks = <&clock 307>;
  

•           clock-names = "iis";
  

...actually, glancing at the driver I'm a little surprised that you don't need to list "i2s_opclk0". Did you test i2s1?

I didn't test i2s1 but i2s_opclk0 is also required. I will resubmit the patch.

...

            pinctrl-names = "default";
            pinctrl-0 = <&i2s1_bus>;
    };

@@ -481,6 +489,8 @@ dmas = <&pdma0 12 &pdma0 11>; dma-names = "tx", "rx";

•           clocks = <&clock 308>;
  

•           clock-names = "iis";
  

...and here.

-Doug

Thanks Padma