[alsa-devel] [PATCH] ASoC: max98090: Add recovery for PLL lock failure

Mark Brown broonie at kernel.org
Wed Sep 3 16:36:18 CEST 2014


On Wed, Sep 03, 2014 at 03:42:48PM +0300, Jarkko Nikula wrote:
> All MAX98090 input clocks MCLK, LRCLK and BCLK must be running and stable
> before powering on the codec in slave mode. Otherwise the PLL may not lock
> to LRCLK causing silence in playback and capture. How often that happens is
> somewhat hardware and clock configuration specific.

I've applied this but I'm thinking it would be more robust to
restructure the interrupt handler so the handling is in a separate
function to the one that reads the status register - that way when
_det_enable_work() polls the status it can just mask out the PLL
interrupt bit and call the usual interrupt handler.  As things stand
we've effectively got two copies of the interrupt code which will
probably cause trouble at some point.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20140903/5fceed71/attachment.sig>


More information about the Alsa-devel mailing list