On Fri, Oct 25, 2019 at 01:09:06PM +0800, Tzung-Bi Shih wrote:
> On Fri, Oct 25, 2019 at 3:14 AM Mark Brown <broonie@kernel.org> wrote:

> > Wow, that's a bit special.  I'm wondering if the PLL unlock error
> > handling isn't connected to the PLL configuration?

> I don't quite understand here.  Did you mean: when max98090_pll_work()
> get called, the PLL may have locked.  The code doesn't check
> M98090_REG_DEVICE_STATUS but shutdown and on anyway.  In the case
> max98090 may generate a new interrupt and again and again?

Something along those lines yeah.