| Message ID | 1533633926-28908-1-git-send-email-j-keerthy@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | clocksource: ti-32k: Add CLOCK_SOURCE_SUSPEND_NONSTOP flag for non-am43 SoCs | expand |
* Keerthy <j-keerthy@ti.com> [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@atomide.com> > Signed-off-by: Keerthy <j-keerthy@ti.com> Acked-by: Tony Lindgren <tony@atomide.com> > --- > 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 > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 8/8/2018 12:28 PM, Tony Lindgren wrote: > * Keerthy <j-keerthy@ti.com> [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@atomide.com> >> Signed-off-by: Keerthy <j-keerthy@ti.com> > > Acked-by: Tony Lindgren <tony@atomide.com> 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 >> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* J, KEERTHY <j-keerthy@ti.com> [180808 13:14]: > > > On 8/8/2018 12:28 PM, Tony Lindgren wrote: > > * Keerthy <j-keerthy@ti.com> [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@atomide.com> > > > Signed-off-by: Keerthy <j-keerthy@ti.com> > > > > Acked-by: Tony Lindgren <tony@atomide.com> > > 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. OK makes sense to me thanks. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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; /*
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> --- drivers/clocksource/timer-ti-32k.c | 3 +++ 1 file changed, 3 insertions(+)