> +static __initdata struct samsung_pll_rate_table epll_24mhz_tbl[] = { > + /* sorted in descending order */ > + /* PLL_36XX_RATE(rate, m, p, s, k) */ > + PLL_36XX_RATE(192000000, 48, 3, 1, 0), > + PLL_36XX_RATE(180633600, 45, 3, 1, 10381), > + PLL_36XX_RATE(180000000, 45, 3, 1, 0), > + PLL_36XX_RATE(73728000, 73, 3, 3, 47710), > + PLL_36XX_RATE(67737600, 90, 4, 3, 20762), > + PLL_36XX_RATE(49152000, 49, 3, 3, 9962), > + PLL_36XX_RATE(45158400, 45, 3, 3, 10381), > + PLL_36XX_RATE(32768000, 131, 3, 5, 4719), > + { }, > +}; I believe the UM says that 64 <= m <= 511. Although the above seems to work on 5250 and 5420, it might be better to use a table that conforms to the constraint on m: PLL_36XX_RATE(192000000, 64, 2, 2, 0), PLL_36XX_RATE(180633600, 90, 3, 2, 20762), PLL_36XX_RATE(180000000, 90, 3, 2, 0), PLL_36XX_RATE(73728000, 98, 2, 4, 19923), PLL_36XX_RATE(67737600, 90, 2, 4, 20762), PLL_36XX_RATE(49152000, 98, 3, 4, 19923), PLL_36XX_RATE(45158400, 90, 3, 4, 20762), PLL_36XX_RATE(32768000, 131, 3, 5, 4719) Thanks, Andrew -- 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