Re: [PATCH v4 1/5] ARC: clockevent: DT based probe

2016-04-14 Thread Daniel Lezcano
On Thu, Apr 14, 2016 at 03:02:38PM +0530, Vineet Gupta wrote: > On Wednesday 13 April 2016 06:29 PM, Daniel Lezcano wrote: > > On Wed, Apr 13, 2016 at 05:10:01PM +0530, Vineet Gupta wrote: > >> - timer frequency is derived from DT (no longer rely on top level > >>DT "clock-frequency" probed ea

Re: [PATCH v4 2/5] ARC: clocksource: DT based probe

2016-04-14 Thread Vineet Gupta
On Thursday 14 April 2016 03:02 PM, Marc Zyngier wrote: >> This is clocksource, not clockevent. cs needs to synchronized across all >> cores so >> > that concurrent gtod call from threads on different cores gives you similar >> > values. This obviously is not true for the local RTC hardware timer.

Re: [PATCH v4 1/5] ARC: clockevent: DT based probe

2016-04-14 Thread Vineet Gupta
On Wednesday 13 April 2016 06:29 PM, Daniel Lezcano wrote: > On Wed, Apr 13, 2016 at 05:10:01PM +0530, Vineet Gupta wrote: >> - timer frequency is derived from DT (no longer rely on top level >>DT "clock-frequency" probed early and exported by asm/clk.h) >> >> - TIMER0_IRQ need not be exporte

Re: [PATCH v4 2/5] ARC: clocksource: DT based probe

2016-04-14 Thread Marc Zyngier
On 14/04/16 10:26, Vineet Gupta wrote: > On Wednesday 13 April 2016 09:52 PM, Marc Zyngier wrote: >>> -int arc_counter_setup(void) +static void __init arc_cs_setup_rtc(struct device_node *node) { - write_aux_reg(ARC_REG_TIMER1_LIMIT, ARC_TIMER_MAX); - write_aux_reg(ARC_REG_TI

Re: [PATCH v4 2/5] ARC: clocksource: DT based probe

2016-04-14 Thread Vineet Gupta
On Wednesday 13 April 2016 09:52 PM, Marc Zyngier wrote: >> -int arc_counter_setup(void) >> > +static void __init arc_cs_setup_rtc(struct device_node *node) >> > { >> > - write_aux_reg(ARC_REG_TIMER1_LIMIT, ARC_TIMER_MAX); >> > - write_aux_reg(ARC_REG_TIMER1_CNT, 0); >> > - write_aux_reg(ARC_RE