Re: [PATCH v3] clk: change clk_ops' -determine_rate() prototype

2015-07-06 Thread Boris Brezillon
On Wed, 3 Jun 2015 16:37:28 -0700 Stephen Boyd sb...@codeaurora.org wrote: On 05/20, Boris Brezillon wrote: Clock rates are stored in an unsigned long field, but -determine_rate() (which returns a rounded rate from a requested one) returns a long value (errors are reported using negative

Re: [PATCH v3] clk: change clk_ops' -determine_rate() prototype

2015-06-04 Thread Boris Brezillon
Hi Stefen, On Wed, 3 Jun 2015 16:37:28 -0700 Stephen Boyd sb...@codeaurora.org wrote: On 05/20, Boris Brezillon wrote: Clock rates are stored in an unsigned long field, but -determine_rate() (which returns a rounded rate from a requested one) returns a long value (errors are reported

Re: [PATCH v3] clk: change clk_ops' -determine_rate() prototype

2015-06-03 Thread Stephen Boyd
On 05/20, Boris Brezillon wrote: Clock rates are stored in an unsigned long field, but -determine_rate() (which returns a rounded rate from a requested one) returns a long value (errors are reported using negative error codes), which can lead to long overflow if the clock rate exceed 2Ghz.