* [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs
@ 2018-08-08 13:14 Keerthy
2018-08-28 11:44 ` J, KEERTHY
2018-08-29 15:17 ` Daniel Lezcano
0 siblings, 2 replies; 4+ messages in thread
From: Keerthy @ 2018-08-08 13:14 UTC (permalink / raw)
To: daniel.lezcano, tglx
Cc: lokeshvutla, linux-kernel, linux-omap, baolin.wang, d-gerlach,
tony, grygorii.strashko, j-keerthy, t-kristo
The 32k clocksource is NONSTOP for non-am43 SoCs. Hence
add the flag for all the other SoCs.
Reported-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Keerthy <j-keerthy@ti.com>
Acked-by: Tony Lindgren <tony@atomide.com>
---
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;
/*
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs
2018-08-08 13:14 [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs Keerthy
@ 2018-08-28 11:44 ` J, KEERTHY
2018-08-28 12:11 ` Daniel Lezcano
2018-08-29 15:17 ` Daniel Lezcano
1 sibling, 1 reply; 4+ messages in thread
From: J, KEERTHY @ 2018-08-28 11:44 UTC (permalink / raw)
To: daniel.lezcano, tglx
Cc: lokeshvutla, linux-kernel, linux-omap, baolin.wang, d-gerlach,
tony, grygorii.strashko, t-kristo
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.
>
> Reported-by: Tony Lindgren <tony@atomide.com>
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> Acked-by: Tony Lindgren <tony@atomide.com>
> ---
>
> 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;
>
> /*
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs
2018-08-28 11:44 ` J, KEERTHY
@ 2018-08-28 12:11 ` Daniel Lezcano
0 siblings, 0 replies; 4+ messages in thread
From: Daniel Lezcano @ 2018-08-28 12:11 UTC (permalink / raw)
To: J, KEERTHY, tglx
Cc: lokeshvutla, linux-kernel, linux-omap, baolin.wang, d-gerlach,
tony, grygorii.strashko, t-kristo
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 <tony@atomide.com>
>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>> Acked-by: Tony Lindgren <tony@atomide.com>
>> ---
>>
>> 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;
>> /*
>>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs
2018-08-08 13:14 [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs Keerthy
2018-08-28 11:44 ` J, KEERTHY
@ 2018-08-29 15:17 ` Daniel Lezcano
1 sibling, 0 replies; 4+ messages in thread
From: Daniel Lezcano @ 2018-08-29 15:17 UTC (permalink / raw)
To: Keerthy, tglx
Cc: lokeshvutla, linux-kernel, linux-omap, baolin.wang, d-gerlach,
tony, grygorii.strashko, t-kristo
On 08/08/2018 15:14, Keerthy wrote:
> The 32k clocksource is NONSTOP for non-am43 SoCs. Hence
> add the flag for all the other SoCs.
>
> Reported-by: Tony Lindgren <tony@atomide.com>
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> Acked-by: Tony Lindgren <tony@atomide.com>
> ---
Applied for 4.19-rc2
Thanks
-- Daniel
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-08-29 15:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-08-08 13:14 [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs Keerthy
2018-08-28 11:44 ` J, KEERTHY
2018-08-28 12:11 ` Daniel Lezcano
2018-08-29 15:17 ` Daniel Lezcano
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).