Re: [PATCH] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs

[J, KEERTHY]

On 8/8/2018 12:28 PM, Tony Lindgren wrote:
> * Keerthy <j-keerthy@xxxxxx> [180807 09:29]:
> > The 32k clocksource is NONSTOP for non-am43 SoCs. Hence
> > add the flag for all the other SoCs.
> >
> > Reported-by: Tony Lindgren <tony@xxxxxxxxxxx>
> > Signed-off-by: Keerthy <j-keerthy@xxxxxx>
>
> Acked-by: Tony Lindgren <tony@xxxxxxxxxxx>

Thanks Tony. Lokesh pointed out about am43x-epos. So will make a minor 
correction to this patch with compatible. I searched all the am4 boards 
& seems like ti,am43 is the most generic compatible cover am43x-epos as 
well. So i will retain your ack and send a v2 with am43 compatible 
changed to a more generic ti,am43.

> > ---
> >   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,am4372"))
> > +		ti_32k_timer.cs.flags |= CLOCK_SOURCE_SUSPEND_NONSTOP;
> > +
> >   	ti_32k_timer.counter = ti_32k_timer.base;
> >   
> >   	/*
> > --
> > 1.9.1