Hi,
On Tue, Mar 12, 2013 at 6:15 AM, Heiko Stübner <heiko@xxxxxxxxx> wrote:
> The common clock framework expects clocks to be prepared
> before they are enabled.
>
> Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx>
> ---
> arch/arm/plat-samsung/samsung-time.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/plat-samsung/samsung-time.c b/arch/arm/plat-samsung/samsung-time.c
> index f899cbc..6ebb851 100644
> --- a/arch/arm/plat-samsung/samsung-time.c
> +++ b/arch/arm/plat-samsung/samsung-time.c
> @@ -355,7 +355,7 @@ static void __init samsung_timer_resources(void)
> if (IS_ERR(timerclk))
> panic("failed to get timers clock for timer");
>
Also how about calling the clk_prepare_enable() only if the clk_get()
call is a sucess? Though clk framework will take care of it. Just
thinking even to avoid that call if clk_get() fails since you are
modifying the code.
> - clk_enable(timerclk);
> + clk_prepare_enable(timerclk);
>
> sprintf(devname, "s3c24xx-pwm.%lu", event_id);
> s3c_device_timer[event_id].id = event_id;
> @@ -369,7 +369,7 @@ static void __init samsung_timer_resources(void)
> if (IS_ERR(tdiv_event))
> panic("failed to get pwm-tdiv clock for event timer");
>
Same here...
> - clk_enable(tin_event);
> + clk_prepare_enable(tin_event);
>
> sprintf(devname, "s3c24xx-pwm.%lu", source_id);
Regards,
Pankaj Jangra
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
