We don't need to enable the clock for pwm everytime after probing. So, this patch moves its clock enabling/disabling into the pwm_{enable,disable}(). Signed-off-by: Huisung Kang <hs1218.kang@xxxxxxxxxxx> --- arch/arm/plat-samsung/pwm.c | 36 +++++++++++++++++++++++++++--------- 1 files changed, 27 insertions(+), 9 deletions(-) diff --git a/arch/arm/plat-samsung/pwm.c b/arch/arm/plat-samsung/pwm.c index c559d84..3a604ca 100644 --- a/arch/arm/plat-samsung/pwm.c +++ b/arch/arm/plat-samsung/pwm.c @@ -110,13 +110,19 @@ int pwm_enable(struct pwm_device *pwm) local_irq_save(flags); - tcon = __raw_readl(S3C2410_TCON); - tcon |= pwm_tcon_start(pwm); - __raw_writel(tcon, S3C2410_TCON); + if (!pwm->running) { + clk_enable(pwm->clk); + clk_enable(pwm->clk_div); + + tcon = __raw_readl(S3C2410_TCON); + tcon |= pwm_tcon_start(pwm); + __raw_writel(tcon, S3C2410_TCON); + + pwm->running = 1; + } local_irq_restore(flags); - pwm->running = 1; return 0; } @@ -129,13 +135,17 @@ void pwm_disable(struct pwm_device *pwm) local_irq_save(flags); - tcon = __raw_readl(S3C2410_TCON); - tcon &= ~pwm_tcon_start(pwm); - __raw_writel(tcon, S3C2410_TCON); + if (pwm->running) { + tcon = __raw_readl(S3C2410_TCON); + tcon &= ~pwm_tcon_start(pwm); + __raw_writel(tcon, S3C2410_TCON); - local_irq_restore(flags); + clk_disable(pwm->clk); + clk_disable(pwm->clk_div); + pwm->running = 0; + } - pwm->running = 0; + local_irq_restore(flags); } EXPORT_SYMBOL(pwm_disable); @@ -185,6 +195,9 @@ int pwm_config(struct pwm_device *pwm, int duty_ns, int period_ns) /* The TCMP and TCNT can be read without a lock, they're not * shared between the timers. */ + clk_enable(pwm->clk); + clk_enable(pwm->clk_div); + tcmp = __raw_readl(S3C2410_TCMPB(pwm->pwm_id)); tcnt = __raw_readl(S3C2410_TCNTB(pwm->pwm_id)); @@ -242,6 +255,9 @@ int pwm_config(struct pwm_device *pwm, int duty_ns, int period_ns) local_irq_restore(flags); + clk_disable(pwm->clk); + clk_disable(pwm->clk_div); + return 0; } @@ -310,6 +326,8 @@ static int s3c_pwm_probe(struct platform_device *pdev) local_irq_restore(flags); + clk_disable(pwm->clk); + clk_disable(pwm->clk_div); ret = pwm_register(pwm); if (ret) { -- 1.7.1 -- 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