RE: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-20 Thread Chris Brandt
Hi Geert, On Friday, January 20, 2017, Geert Uytterhoeven wrote: > It takes longer to boot, though. I guess due to more kernel output? You can see there is a jump in the time when the system clocksource changes from mtu2 to ostm. [0.079591] clocksource: Switched to clocksource ostm [0.33

Re: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-19 Thread Geert Uytterhoeven
Hi Chris, On Fri, Jan 20, 2017 at 4:56 AM, Chris Brandt wrote: > On Monday, January 16, 2017, Geert Uytterhoeven wrote: >> As they're independent channels, it doesn't matter which one is used for >> which function, right? >> >> You could use the first probed channel for the most important functio

RE: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-19 Thread Chris Brandt
Hi Geert, On Monday, January 16, 2017, Geert Uytterhoeven wrote: > As they're independent channels, it doesn't matter which one is used for > which function, right? > > You could use the first probed channel for the most important function > (clocksource?), and the second one for the other functi

RE: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-16 Thread Chris Brandt
Hi Geert, On Monday, January 16, 2017, Geert Uytterhoeven wrote: > > Do you think it would be better if a driver instance only does 1 > > thing: Either 'clocksource' or 'clock event'?? > > Then, I would make 2 ostm nodes and pass in the mode I would like it > operate in? > > > > For example: > > >

Re: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-16 Thread Geert Uytterhoeven
Hi Chris, On Sat, Jan 14, 2017 at 4:30 AM, Chris Brandt wrote: > On Friday, January 13, 2017, Geert Uytterhoeven wrote: >> > +The OSTM comes with 2 independent channels. >> > +We will use the first channel (OSTM0) as a free running clocksource >> > +and the second channel (OSTM1) as a interrupt d

Re: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-16 Thread Simon Horman
On Sat, Jan 14, 2017 at 03:30:36AM +, Chris Brandt wrote: > Hi Geert, > > Thank you for your review. > > > On Friday, January 13, 2017, Geert Uytterhoeven wrote: > > > +The OSTM comes with 2 independent channels. > > > +We will use the first channel (OSTM0) as a free running clocksource > >

RE: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-13 Thread Chris Brandt
Hi Geert, Thank you for your review. On Friday, January 13, 2017, Geert Uytterhoeven wrote: > > +The OSTM comes with 2 independent channels. > > +We will use the first channel (OSTM0) as a free running clocksource > > +and the second channel (OSTM1) as a interrupt driven clock event. > > + > > +

Re: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-13 Thread Geert Uytterhoeven
Hi Chris, On Thu, Jan 12, 2017 at 7:59 PM, Chris Brandt wrote: > Signed-off-by: Chris Brandt > --- > .../devicetree/bindings/timer/renesas,ostm.txt | 36 > ++ > 1 file changed, 36 insertions(+) > create mode 100644 Documentation/devicetree/bindings/timer/renesas,ostm.t