| Message ID | 1533734099-1398-1-git-send-email-j-keerthy@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs | expand |
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; > > /* >
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; >> /* >>
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
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; /*