Re: [PATCH 07/22] clk: sunxi: Fix PLL6 calculation on sun6i

From: Chen-Yu Tsai
Date: Fri May 23 2014 - 10:44:12 EST


Hi,

On Fri, May 23, 2014 at 9:09 PM, Emilio LÃpez <emilio@xxxxxxxxxxxxx> wrote:
> Hi,
>
> El 23/05/14 04:51, Chen-Yu Tsai escribiÃ:
>
>> The N factor for PLL6 counts from 1 to 32, as specified in the A23
>> manual, and shown in Allwinner's original code.
>
>
> Are you sure about this? The A23 manual I'm looking at has a 5-bit field (so
> 0-31) on 12:8, and the examples count from 0 (eg Factor=0 N=1 ... Factor=31
> N=32).

>From the A23 manual:

Factor = 0, N = 1
Factor = 1, N = 2
Factor = 2, N = 3
and so on


>From the A20 manual:

Factor = 0, N = 0
Factor = 1, N = 1
Factor = 2, N = 2

The "N factor" I referred to in my commit message is the N used in the
calculation,
not what is stored in the bit field.

Hope this makes it clear. :)

Cheers
ChenYu
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/