linux-kernel - Re: [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <12b80b5f-3dd8-793e-75b2-94d2a0248d3e@ti.com>
Date:   Tue, 28 Aug 2018 17:14:00 +0530
From:   "J, KEERTHY" <j-keerthy@...com>
To:     <daniel.lezcano@...aro.org>, <tglx@...utronix.de>
CC:     <lokeshvutla@...com>, <linux-kernel@...r.kernel.org>,
        <linux-omap@...r.kernel.org>, <baolin.wang@...aro.org>,
        <d-gerlach@...com>, <tony@...mide.com>, <grygorii.strashko@...com>,
        <t-kristo@...com>
Subject: Re: [PATCH v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP
 flag for non-am43 SoCs



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@...mide.com>
> Signed-off-by: Keerthy <j-keerthy@...com>
> Acked-by: Tony Lindgren <tony@...mide.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;
>   
>   	/*
>