Re: [PATCH v2 04/15] clk: divider: Make generic for usage elsewhere

[Srinivas Kandagatla]

On 07/10/14 19:26, Stephen Boyd wrote:
> On 10/07/2014 10:27 AM, Srinivas Kandagatla wrote:
> > Hi Stephen,
> >
> > Just noticed this regression while testing the patch on Arndale board.
> >
> > https://bugs.linaro.org/show_bug.cgi?id=740
>
> If you return value it works correctly right?
I think you are right, ..
From last message from bugs.linaro.org:

"
That seems to be it!

This change:
--- a/drivers/clk/clk-divider.c
+++ b/drivers/clk/clk-divider.c
@@ -366,7 +366,7 @@ int divider_get_val(unsigned long rate, unsigned long
parent

        min_t(unsigned int, value, div_mask(width));

-       return 0;
+       return value;
 }
 EXPORT_SYMBOL_GPL(divider_get_val);

- makes Arndale to boot; rootfs on mmc mounts OK.
"

thanks,
srini
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/