From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs Date: Tue, 28 Aug 2018 14:11:13 +0200 Message-ID: <7234a135-96e4-89d8-1b3f-c2bb4e5534f5@linaro.org> References: <1533734099-1398-1-git-send-email-j-keerthy@ti.com> <12b80b5f-3dd8-793e-75b2-94d2a0248d3e@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <12b80b5f-3dd8-793e-75b2-94d2a0248d3e@ti.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: "J, KEERTHY" , tglx@linutronix.de Cc: lokeshvutla@ti.com, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, baolin.wang@linaro.org, d-gerlach@ti.com, tony@atomide.com, grygorii.strashko@ti.com, t-kristo@ti.com List-Id: linux-omap@vger.kernel.org On 28/08/2018 13:44, J, KEERTHY wrote: > > > On 8/8/2018 6:44 PM, Keerthy wrote: >> The 32k clocksource is NONSTOP for non-am43 SoCs. Hence >> add the flag for all the other SoCs. > > A gentle ping on this one. Yeah, saw it. Returning from vacation, still unstacking emails ... :) >> Reported-by: Tony Lindgren >> Signed-off-by: Keerthy >> Acked-by: Tony Lindgren >> --- >> >> Changes in v2: >> >>    * Changed am43 compatible to more generic ti,am43 to cover epos >> boards. >>    * Added Tony's Ack. >> >>   drivers/clocksource/timer-ti-32k.c | 3 +++ >>   1 file changed, 3 insertions(+) >> >> diff --git a/drivers/clocksource/timer-ti-32k.c >> b/drivers/clocksource/timer-ti-32k.c >> index 29e2e1a..dd09171 100644 >> --- a/drivers/clocksource/timer-ti-32k.c >> +++ b/drivers/clocksource/timer-ti-32k.c >> @@ -97,6 +97,9 @@ static int __init ti_32k_timer_init(struct >> device_node *np) >>           return -ENXIO; >>       } >>   +    if (!of_machine_is_compatible("ti,am43")) >> +        ti_32k_timer.cs.flags |= CLOCK_SOURCE_SUSPEND_NONSTOP; >> + >>       ti_32k_timer.counter = ti_32k_timer.base; >>         /* >> -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog