From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751726Ab3B0Ffb (ORCPT ); Wed, 27 Feb 2013 00:35:31 -0500 Received: from mail-pa0-f47.google.com ([209.85.220.47]:59865 "EHLO mail-pa0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750793Ab3B0Ff3 convert rfc822-to-8bit (ORCPT ); Wed, 27 Feb 2013 00:35:29 -0500 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT To: Eduardo Valentin , From: Mike Turquette In-Reply-To: <1361919218-9788-16-git-send-email-eduardo.valentin@ti.com> Cc: , , , Eduardo Valentin References: <1361919218-9788-1-git-send-email-eduardo.valentin@ti.com> <1361919218-9788-16-git-send-email-eduardo.valentin@ti.com> Message-ID: <20130227053523.14870.20885@quantum> User-Agent: alot/0.3.3+ Subject: Re: [PATCH 15/15] staging: omap-thermal: update clock prepare count Date: Tue, 26 Feb 2013 21:35:23 -0800 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Eduardo Valentin (2013-02-26 14:53:38) > This patch changes the clock management code to also update > the clock prepare counter, this way we won't skip the enable/disable > operation due to prepare dependencies. > > Signed-off-by: Eduardo Valentin Hi Eduardo, I didn't look through the other patches in this series (or even the existing driver that is merged). With that said, I have a question: does this driver aggressively call clk_enable/clk_disable in call sites other than those listed in this patch? If so it might be a good idea to call clk_prepare_enable and clk_disable_unprepare in those location. Of course only call it if the context can sleep (e.g. not an interrupt handler). If the driver doesn't aggressively call enable/disable elsewhere than you can disregard my message ;-) Regards, Mike > --- > drivers/staging/omap-thermal/omap-bandgap.c | 10 +++++----- > 1 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/omap-thermal/omap-bandgap.c b/drivers/staging/omap-thermal/omap-bandgap.c > index 83f74f4..d4a3788 100644 > --- a/drivers/staging/omap-thermal/omap-bandgap.c > +++ b/drivers/staging/omap-thermal/omap-bandgap.c > @@ -943,7 +943,7 @@ int omap_bandgap_probe(struct platform_device *pdev) > > bg_ptr->clk_rate = clk_rate; > if (OMAP_BANDGAP_HAS(bg_ptr, CLK_CTRL)) > - clk_enable(bg_ptr->fclock); > + clk_prepare_enable(bg_ptr->fclock); > > > mutex_init(&bg_ptr->bg_mutex); > @@ -1013,7 +1013,7 @@ int omap_bandgap_probe(struct platform_device *pdev) > > disable_clk: > if (OMAP_BANDGAP_HAS(bg_ptr, CLK_CTRL)) > - clk_disable(bg_ptr->fclock); > + clk_disable_unprepare(bg_ptr->fclock); > put_clks: > clk_put(bg_ptr->fclock); > clk_put(bg_ptr->div_clk); > @@ -1044,7 +1044,7 @@ int omap_bandgap_remove(struct platform_device *pdev) > omap_bandgap_power(bg_ptr, false); > > if (OMAP_BANDGAP_HAS(bg_ptr, CLK_CTRL)) > - clk_disable(bg_ptr->fclock); > + clk_disable_unprepare(bg_ptr->fclock); > clk_put(bg_ptr->fclock); > clk_put(bg_ptr->div_clk); > > @@ -1143,7 +1143,7 @@ static int omap_bandgap_suspend(struct device *dev) > omap_bandgap_power(bg_ptr, false); > > if (OMAP_BANDGAP_HAS(bg_ptr, CLK_CTRL)) > - clk_disable(bg_ptr->fclock); > + clk_disable_unprepare(bg_ptr->fclock); > > return err; > } > @@ -1153,7 +1153,7 @@ static int omap_bandgap_resume(struct device *dev) > struct omap_bandgap *bg_ptr = dev_get_drvdata(dev); > > if (OMAP_BANDGAP_HAS(bg_ptr, CLK_CTRL)) > - clk_enable(bg_ptr->fclock); > + clk_prepare_enable(bg_ptr->fclock); > > omap_bandgap_power(bg_ptr, true); > > -- > 1.7.7.1.488.ge8e1c > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html