Re: [PATCH 3/3] ARM: da850: fix da850_set_pll0rate()

2016-12-02 Thread Bartosz Golaszewski
2016-12-02 12:20 GMT+01:00 Sekhar Nori : > On Thursday 01 December 2016 10:45 PM, Bartosz Golaszewski wrote: >> This function is broken - its second argument is an index to the freq >> table, not the requested clock rate in Hz. It leads to an oops when >> called from clk_set_rate() since this argum

Re: [PATCH 3/3] ARM: da850: fix da850_set_pll0rate()

2016-12-02 Thread Sekhar Nori
On Thursday 01 December 2016 10:45 PM, Bartosz Golaszewski wrote: > This function is broken - its second argument is an index to the freq > table, not the requested clock rate in Hz. It leads to an oops when > called from clk_set_rate() since this argument isn't bounds checked > either. > > Fix it

[PATCH 3/3] ARM: da850: fix da850_set_pll0rate()

2016-12-01 Thread Bartosz Golaszewski
This function is broken - its second argument is an index to the freq table, not the requested clock rate in Hz. It leads to an oops when called from clk_set_rate() since this argument isn't bounds checked either. Fix it by iterating over the array of supported frequencies and selecting a one that