[alsa-devel] [PATCH] ASoC: sgtl5000: Allow the codec to work in slave mode

[Fabio Estevam]

Hi Mark,

On Wed, Oct 1, 2014 at 3:27 PM, Mark Brown <broonie at kernel.org> wrote:
> On Tue, Sep 30, 2014 at 09:38:21PM +0200, Michael Trimarchi wrote:
>> On Tue, Sep 30, 2014 at 9:29 PM, Mark Brown <broonie at kernel.org> wrote:
>
>> >> Any particular register I should monitor?
>
>> > Did you just listen or did you measure the performance?
>
>> >> Sorry, but I guess I did not understand your last question.
>
>> > I'm really not sure how to simplify it...  clearly we're skipping some
>> > configuration here, what does it do - is it purely for generating BCLK
>> > and LRCLK?
>
>> Are you talking about clock_in and sigma-delta? So no  the bclk and lrclk but
>> the clkin of the codec.
>
> I'm sorry but I can't parse what you're saying terribly clearly - please
> bear in mind that I don't know anything about this device, that's one of
> the reasons I'm having to ask questions here.  If the configuration that
> is being skipped is not about generating BCLK and LRCLK then it seems
> like it's going to be wanted in slave mode as well as master.

This patch does not skip any configuration. It just remove the error
case for sgtl5000 in slave mode.

Not sure why the error case was there in the first place, as I think
sgtl5000 slave mode has never been tested before.