Re: [PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-05 Thread Roman Volkov
В Tue, 5 Jan 2016 11:31:37 +0100 Daniel Lezcano пишет: > On 01/05/2016 11:00 AM, Russell King - ARM Linux wrote: > > On Tue, Jan 05, 2016 at 12:42:42PM +0300, Roman Volkov wrote: > >> Why multiply by two? Good question. Maybe there is a reserve for > >> stability. The value passed by the system

Re: [PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-05 Thread Daniel Lezcano
On 01/05/2016 11:00 AM, Russell King - ARM Linux wrote: On Tue, Jan 05, 2016 at 12:42:42PM +0300, Roman Volkov wrote: Why multiply by two? Good question. Maybe there is a reserve for stability. The value passed by the system to the set_next_event() should be not lesser than this value, and theor

Re: [PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-05 Thread Daniel Lezcano
On 01/05/2016 10:42 AM, Roman Volkov wrote: В Tue, 5 Jan 2016 10:01:07 +0100 Daniel Lezcano пишет: On 01/01/2016 02:24 PM, Roman Volkov wrote: From: Roman Volkov The vt8500 clocksource driver declares itself as capable to handle the minimum delay of 4 cycles by passing the value into clocke

Re: [PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-05 Thread Russell King - ARM Linux
On Tue, Jan 05, 2016 at 12:42:42PM +0300, Roman Volkov wrote: > Why multiply by two? Good question. Maybe there is a reserve for > stability. The value passed by the system to the set_next_event() should > be not lesser than this value, and theoretically, we should not > multiply MIN_OSCR_DELTA by

Re: [PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-05 Thread Roman Volkov
В Tue, 5 Jan 2016 10:01:07 +0100 Daniel Lezcano пишет: > On 01/01/2016 02:24 PM, Roman Volkov wrote: > > From: Roman Volkov > > > > The vt8500 clocksource driver declares itself as capable to handle > > the minimum delay of 4 cycles by passing the value into > > clockevents_config_and_register()

Re: [PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-05 Thread Daniel Lezcano
On 01/01/2016 02:24 PM, Roman Volkov wrote: From: Roman Volkov The vt8500 clocksource driver declares itself as capable to handle the minimum delay of 4 cycles by passing the value into clockevents_config_and_register(). The vt8500_timer_set_next_event() requires the passed cycles value to be a

[PATCH v3 1/3] clocksource/vt8500: Increase the minimum delta

2016-01-01 Thread Roman Volkov
From: Roman Volkov The vt8500 clocksource driver declares itself as capable to handle the minimum delay of 4 cycles by passing the value into clockevents_config_and_register(). The vt8500_timer_set_next_event() requires the passed cycles value to be at least 16. The impact is that userspace hangs