Re: [PATCH 1/4 v4] ASoC: add a WM8978 codec driver

[Mark Brown <broonie@opensource.wolfsonmicro.com>]

On Fri, Jan 29, 2010 at 02:57:34PM +0100, Guennadi Liakhovetski wrote:
> On Wed, 27 Jan 2010, Mark Brown wrote:

> > I'm fairly sure this and the similar logic for SYSCLK can be squashed
> > together with some suitable local variables.  Might be more legible
> > since this requires some staring at.  I didn't actually go so far as to
> > work out what the relevant code is, though.

> Well, not really. In one case f_PLLOUT can be derived directly, because 
> OPCLKDIV covers the whole its value range 1-4, whereas MCLKDIV takes 
> values 1, 3/2, 2, 3, 4, 6, 8, 12, so, you have to apply an iterative 
> process to select the best match. I've just sent a patch, that improves 

I don't follow your logic there at all, I'm afraid.  Both options have a
table of possible values for the divider (and hence the PLL output
frequencies they can use) which they need to additionally constrain
based on the PLL operating conditions.

Looking at the code I think half the issue here is that all the
constraints that are shared from the PLL and its input path aren't
factored out but are in per-output conditional cases, making those more
complex than they need to be.  The magic number heaviness of the code
and comments accentuates the issue.