Re: [PATCH v4 2/2] clocksource: driver for Conexant Digicolor SoC timer

2015-01-26 Thread Thomas Gleixner
On Mon, 26 Jan 2015, Baruch Siach wrote: > Hi Daniel, > > On Mon, Jan 26, 2015 at 10:43:43AM +0100, Daniel Lezcano wrote: > > What do you think ? > > OK. I'll give it a try. > > Thanks for your prompt response and thorough review. Can you please trim your replies proper? It's annoying to page t

Re: [PATCH v4 2/2] clocksource: driver for Conexant Digicolor SoC timer

2015-01-26 Thread Daniel Lezcano
On 01/26/2015 11:15 AM, Baruch Siach wrote: Hi Daniel, [ ... ] What do you think ? OK. I'll give it a try. Thanks for your prompt response and thorough review. Maxime Ripard just sent a patchset (patch 4/5) with some changes suggested above :) http://www.spinics.net/lists/arm-kernel/

Re: [PATCH v4 2/2] clocksource: driver for Conexant Digicolor SoC timer

2015-01-26 Thread Baruch Siach
Hi Daniel, On Mon, Jan 26, 2015 at 10:43:43AM +0100, Daniel Lezcano wrote: [...] > >+static struct dc_dev_t { > >+void __iomem *timer_base; > >+u32 ticks_per_jiffy; > >+} dc_dev; > > Hi Baruch, > > your code is valid but I think there is a misunderstanding when we talked > about the enca

Re: [PATCH v4 2/2] clocksource: driver for Conexant Digicolor SoC timer

2015-01-26 Thread Daniel Lezcano
On 01/26/2015 08:56 AM, Baruch Siach wrote: Add clocksource driver to the Conexant CX92755 SoC, part of the Digicolor SoCs series. Hardware provides 8 timers, A to H. Timer A is dedicated to a future watchdog driver so we don't use it here. Use timer B for sched_clock, and timer C for clock_event

[PATCH v4 2/2] clocksource: driver for Conexant Digicolor SoC timer

2015-01-25 Thread Baruch Siach
Add clocksource driver to the Conexant CX92755 SoC, part of the Digicolor SoCs series. Hardware provides 8 timers, A to H. Timer A is dedicated to a future watchdog driver so we don't use it here. Use timer B for sched_clock, and timer C for clock_event. Signed-off-by: Baruch Siach --- drivers/c