Re: RE: [PATCH] clk: samsung: Fix pll36xx_recalc_rate to handle kdiv properly

2013-06-10 Thread Mike Turquette
Quoting Kukjin Kim (2013-06-05 04:51:29) > Doug Anderson wrote: > > > > The KDIV value is often listed as unsigned but it needs to be treated > > as a 16-bit signed value when using it in calculations. Fix our rate > > recalculation to do this correctly. > > > > Before doing this, I tried settin

RE: [PATCH] clk: samsung: Fix pll36xx_recalc_rate to handle kdiv properly

2013-06-05 Thread Kukjin Kim
Doug Anderson wrote: > > The KDIV value is often listed as unsigned but it needs to be treated > as a 16-bit signed value when using it in calculations. Fix our rate > recalculation to do this correctly. > > Before doing this, I tried setting EPLL on exynos5250 to: > rate, m, p, s, k = 800

Re: [PATCH] clk: samsung: Fix pll36xx_recalc_rate to handle kdiv properly

2013-05-31 Thread Vikas Sajjan
Hi Doug, Looks good to me. On 1 June 2013 00:28, Doug Anderson wrote: > The KDIV value is often listed as unsigned but it needs to be treated > as a 16-bit signed value when using it in calculations. Fix our rate > recalculation to do this correctly. > > Before doing this, I tried setting EPLL

[PATCH] clk: samsung: Fix pll36xx_recalc_rate to handle kdiv properly

2013-05-31 Thread Doug Anderson
The KDIV value is often listed as unsigned but it needs to be treated as a 16-bit signed value when using it in calculations. Fix our rate recalculation to do this correctly. Before doing this, I tried setting EPLL on exynos5250 to: rate, m, p, s, k = 8000, 107, 2, 4, 43691 This rate is ex