On 06/09/2017 05:59 AM, Chanwoo Choi wrote:
--- a/drivers/clk/samsung/clk-exynos5420.c +++ b/drivers/clk/samsung/clk-exynos5420.c @@ -1280,2 +1280,2 @@ static void __init exynos5420_clk_sleep_init(void) {} PLL_35XX_RATE(200000000, 200, 3, 3), };I got a following frequency(Recalc rate) value. I think there are two much different value for 400000000U and 45158400U. fin_pll Recalc rate mdiv pdiv sdiv kdiv 24000000 600000000 100 2 1 0 24000000 400000000 200 3 2 0 24000000 394784008.5 197 3 2 25690 24000000 361267219 301 5 2 3671 24000000 200000000 200 3 3 0 24000000 196607995.7 197 3 3 -25690 24000000 180633609.5 301 5 3 3671 24000000 131072007.3 131 3 3 4719 24000000 100000000 200 3 4 0 24000000 65536003.66 131 3 4 4719 24000000 49348001.07 197 3 5 25690 24000000 180633609.5 301 5 3 3671 24000000 32768001.83 131 3 5 4719+static const struct samsung_pll_rate_table exynos5420_epll_24mhz_tbl[] = { + PLL_36XX_RATE(600000000U, 100, 2, 1, 0), + PLL_36XX_RATE(400000000U, 200, 2, 2, 0),
For 400MHz, pdiv is 3 instead of 2 as following: PLL_36XX_RATE(400000000U, 200, 3, 2, 0),
Thanks a lot for testing this, I originally just verified 131072000U, 180633600U, 196608000U values. I'll amend this entry and will post
v2 of the patch series shortly.
+ PLL_36XX_RATE(393216000U, 197, 3, 2, 25690), + PLL_36XX_RATE(361267200U, 301, 5, 2, 3671), + PLL_36XX_RATE(200000000U, 200, 3, 3, 0), + PLL_36XX_RATE(196608000U, 197, 3, 3, -25690), + PLL_36XX_RATE(180633600U, 301, 5, 3, 3671), + PLL_36XX_RATE(131072000U, 131, 3, 3, 4719), + PLL_36XX_RATE(100000000U, 200, 3, 4, 0), + PLL_36XX_RATE(65536000U, 131, 3, 4, 4719), + PLL_36XX_RATE(49152000U, 197, 3, 5, 25690), + PLL_36XX_RATE(45158400U, 301, 5, 3, 3671),
>
I got the fout value (180633609) instead of 45158400U when I used the M(301) P(5) S(3) K(3671).
>
fin_pll Recalc rate mdiv pdiv sdiv kdiv 24000000 180633609 301 5 3 3671
I'm going to drop this entry as it is not really needed now.
+ PLL_36XX_RATE(32768000U, 131, 3, 5, 4719), +};
-- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
