Re: [PATCH v2] clocksource/drivers/tango_xtal: Add new timer for Tango SoCs

2015-10-08 Thread Mason
On 07/10/2015 18:03, Daniel Lezcano wrote: > On 10/07/2015 03:17 PM, Marc Gonzalez wrote: > >> I'm also confused that you've replaced panic() with pr_err/return. >> AFAIU, if I don't have a clocksource/sched_clock, the system is dead >> in the water. Might as well stop there, and wait for the oper

Re: [PATCH v2] clocksource/drivers/tango_xtal: Add new timer for Tango SoCs

2015-10-07 Thread Daniel Lezcano
On 10/07/2015 03:17 PM, Marc Gonzalez wrote: On 07/10/2015 14:31, Daniel Lezcano wrote: On 10/07/2015 01:35 PM, Marc Gonzalez wrote: Sigma Designs Tango platforms provide a 27 MHz crystal oscillator. Use it for clocksource, sched_clock, and delay_timer. Signed-off-by: Marc Gonzalez --- AFAICS

Re: [PATCH v2] clocksource/drivers/tango_xtal: Add new timer for Tango SoCs

2015-10-07 Thread Marc Gonzalez
On 07/10/2015 14:31, Daniel Lezcano wrote: > On 10/07/2015 01:35 PM, Marc Gonzalez wrote: >> Sigma Designs Tango platforms provide a 27 MHz crystal oscillator. >> Use it for clocksource, sched_clock, and delay_timer. >> >> Signed-off-by: Marc Gonzalez >> --- >> AFAICS, clocksource_register_hz does

Re: [PATCH v2] clocksource/drivers/tango_xtal: Add new timer for Tango SoCs

2015-10-07 Thread Daniel Lezcano
On 10/07/2015 01:35 PM, Marc Gonzalez wrote: Sigma Designs Tango platforms provide a 27 MHz crystal oscillator. Use it for clocksource, sched_clock, and delay_timer. Signed-off-by: Marc Gonzalez --- AFAICS, clocksource_register_hz does not report failures via its return value (always 0) but wri