On 24 February 2015 at 10:39, Heiko Stübner <[email protected]> wrote: > Commit bca9690b9426 ("clk: divider: Make generic for usage elsewhere") > returned only the divider value for read-only dividers instead of the > actual rate. > > Fixes: bca9690b9426 ("clk: divider: Make generic for usage elsewhere") > Signed-off-by: Heiko Stuebner <[email protected]>
I just created an identical patch, so Reviewed-by: James Hogan <[email protected]> Tested-by: James Hogan <[email protected]> Thanks James > --- > drivers/clk/clk-divider.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/clk-divider.c b/drivers/clk/clk-divider.c > index a213455..5b4c018 100644 > --- a/drivers/clk/clk-divider.c > +++ b/drivers/clk/clk-divider.c > @@ -355,7 +355,7 @@ static long clk_divider_round_rate(struct clk_hw *hw, > unsigned long rate, > bestdiv = readl(divider->reg) >> divider->shift; > bestdiv &= div_mask(divider->width); > bestdiv = _get_div(divider->table, bestdiv, divider->flags); > - return bestdiv; > + return DIV_ROUND_UP(*prate, bestdiv); > } > > return divider_round_rate(hw, rate, prate, divider->table, > -- > 2.1.4 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
