Re: [PATCH 2/5] ASoC: da7213: Add checking of SRM lock status before enabling DAI

From: Mark Brown
Date: Tue May 10 2016 - 14:24:05 EST


On Tue, May 10, 2016 at 07:21:43PM +0100, Mark Brown wrote:
> On Tue, May 10, 2016 at 04:11:05PM +0100, Adam Thomson wrote:
>
> > + /* Slave mode, if SRM not enabled no need for status checks */
> > + pll_ctrl = snd_soc_read(codec, DA7213_PLL_CTRL);
> > + if (!(pll_ctrl & DA7213_PLL_SRM_EN))
> > + return 0;
>
> This isn't going to work, the driver has no volatile registers but it
> does have a cache...

Ugh, yes it does - was looking at the wrong driver there.

Attachment: signature.asc
Description: PGP signature