From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Ferre Subject: Re: [PATCH 4/5] rtc: at91sam9: retain slow clock and check its rate Date: Tue, 9 Sep 2014 10:36:39 +0200 Message-ID: <540EBC17.50407@atmel.com> References: <1409733934-14465-1-git-send-email-boris.brezillon@free-electrons.com> <1409733934-14465-5-git-send-email-boris.brezillon@free-electrons.com> <20140908173338.GD14623@piout.net> <20140908212218.6a81af86@bbrezillon> <20140908213743.27d7a1bc@bbrezillon> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140908213743.27d7a1bc@bbrezillon> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Boris BREZILLON Cc: Alexandre Belloni , Jean-Christophe Plagniol-Villard , Andrew Victor , Alessandro Zummo , rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 08/09/2014 21:37, Boris BREZILLON : > On Mon, 8 Sep 2014 21:22:18 +0200 > Boris BREZILLON wrote: > >> On Mon, 8 Sep 2014 19:33:38 +0200 >> Alexandre Belloni wrote: >> >>> On 03/09/2014 at 10:45:33 +0200, Boris Brezillon wrote : >>>> The RTT block is using the slow clock and expect it to run at 32KHz. >>>> Now that we moved to the CCF it's better to retain the clk reference so >>>> that the CCF can't disable the slow clock considering it is unused. >>>> >>>> Signed-off-by: Boris BREZILLON >>>> --- >>>> drivers/rtc/rtc-at91sam9.c | 24 ++++++++++++++++++++++++ >>>> 1 file changed, 24 insertions(+) >>>> >>>> diff --git a/drivers/rtc/rtc-at91sam9.c b/drivers/rtc/rtc-at91sam9.c >>>> index 57014b7..5c5093b 100644 >>>> --- a/drivers/rtc/rtc-at91sam9.c >>>> +++ b/drivers/rtc/rtc-at91sam9.c >>>> @@ -21,6 +21,7 @@ >>>> #include >>>> #include >>>> #include >>>> +#include >>>> >>>> /* >>>> * This driver uses two configurable hardware resources that live in the >>>> @@ -74,6 +75,7 @@ struct sam9_rtc { >>>> u32 imr; >>>> void __iomem *gpbr; >>>> int irq; >>>> + struct clk *sclk; >>>> }; >>>> >>>> #define rtt_readl(rtc, field) \ >>>> @@ -373,6 +375,25 @@ static int at91_rtc_probe(struct platform_device *pdev) >>>> return ret; >>>> } >>>> >>>> + /* Retain slow clk if it is specified in the DT. >>>> + * Do not complain if slow clk is missing, but check its rate >>>> + * if it is available. >>>> + */ >>>> + rtc->sclk = devm_clk_get(&pdev->dev, NULL); >>>> + if (!IS_ERR(rtc->sclk)) { >>>> + if (clk_get_rate(rtc->sclk) != AT91_SLOW_CLOCK) { >>> >>> I would not bother doing that check but use the value for MR instead of >>> AT91_SLOW_CLOCK (see my previous mail). >> >> Unfortunately, we can't get rid of this macro without modifying the >> clk_lookup table in several arch/arm/mach-at91/.c files in >> order to handle non DT/CCF cases (which will remain until all non DT >> boards are moved to DT). > > After taking a closer look at what should be modified, I think it's > worth it: there's only 5 impacted files (at91sam9260.c, at91sam9261.c, > at91sam9263.c, at91sam9rl.c and at91sam9g45.c) and adding a clk_lookup > entry is pretty easy. > > Moreover we'll end up with a clean driver and won't have to bother > about cleaning it up when dropping non DT boards support. I vote for this => +1 ;-) Bye, -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html