Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 30, 2017 at 11:53:35AM +0200, Linus Walleij wrote: > On Tue, May 30, 2017 at 10:54 AM, Daniel Lezcano > wrote: > > On 30/05/2017 09:44, Linus Walleij wrote: > > >> I guess what we can do is make two different sched_clock() > >> callbacks: one for upward and

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 30, 2017 at 11:53:35AM +0200, Linus Walleij wrote: > On Tue, May 30, 2017 at 10:54 AM, Daniel Lezcano > wrote: > > On 30/05/2017 09:44, Linus Walleij wrote: > > >> I guess what we can do is make two different sched_clock() > >> callbacks: one for upward and one for downward counting.

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 30, 2017 at 10:54 AM, Daniel Lezcano wrote: > On 30/05/2017 09:44, Linus Walleij wrote: >> I guess what we can do is make two different sched_clock() >> callbacks: one for upward and one for downward counting. >> >> Would you like an optimization like that?

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 30, 2017 at 10:54 AM, Daniel Lezcano wrote: > On 30/05/2017 09:44, Linus Walleij wrote: >> I guess what we can do is make two different sched_clock() >> callbacks: one for upward and one for downward counting. >> >> Would you like an optimization like that? >> >> (It makes sense.) >

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On 30/05/2017 09:44, Linus Walleij wrote: > On Sun, May 28, 2017 at 8:27 PM, Daniel Lezcano > wrote: >> On 28/05/2017 15:59, Linus Walleij wrote: >>> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: >>> Also clean up space-before-tab issues

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On 30/05/2017 09:44, Linus Walleij wrote: > On Sun, May 28, 2017 at 8:27 PM, Daniel Lezcano > wrote: >> On 28/05/2017 15:59, Linus Walleij wrote: >>> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: >>> Also clean up space-before-tab issues in the documentation.

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Sun, May 28, 2017 at 8:27 PM, Daniel Lezcano wrote: > On 28/05/2017 15:59, Linus Walleij wrote: >> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: >> >>> Also clean up space-before-tab issues in the documentation. >>> >>> Signed-off-by:

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Sun, May 28, 2017 at 8:27 PM, Daniel Lezcano wrote: > On 28/05/2017 15:59, Linus Walleij wrote: >> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: >> >>> Also clean up space-before-tab issues in the documentation. >>> >>> Signed-off-by: Andrew Jeffery >> >> Reviewed-by: Linus Walleij

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On 28/05/2017 15:59, Linus Walleij wrote: > On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: > >> Also clean up space-before-tab issues in the documentation. >> >> Signed-off-by: Andrew Jeffery > > Reviewed-by: Linus Walleij >

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On 28/05/2017 15:59, Linus Walleij wrote: > On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: > >> Also clean up space-before-tab issues in the documentation. >> >> Signed-off-by: Andrew Jeffery > > Reviewed-by: Linus Walleij > > Does this collide with Daniel's 2500 patch? > > Sorry

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Thu, May 25, 2017 at 10:28 PM, Daniel Lezcano wrote: > Shouldn't the compatible string be: > > "aspeed,ast2400-timer", "faraday,fttmr010" > "aspeed,ast2500-timer", "faraday,fttmr010" Actually not this time. The Gemini and Moxart timers are

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Thu, May 25, 2017 at 10:28 PM, Daniel Lezcano wrote: > Shouldn't the compatible string be: > > "aspeed,ast2400-timer", "faraday,fttmr010" > "aspeed,ast2500-timer", "faraday,fttmr010" Actually not this time. The Gemini and Moxart timers are compatible with the pure Faraday

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. > > Signed-off-by: Andrew Jeffery Reviewed-by: Linus Walleij Does this collide with Daniel's 2500 patch? Sorry for

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. > > Signed-off-by: Andrew Jeffery Reviewed-by: Linus Walleij Does this collide with Daniel's 2500 patch? Sorry for the mess, tell me if I need to fix something up :( Yours,

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Thu, 2017-05-25 at 22:28 +0200, Daniel Lezcano wrote: > On Tue, May 16, 2017 at 03:58:40PM +0800, Andrew Jeffery wrote: > > Also clean up space-before-tab issues in the documentation. > > Andrew, > > I reworked the patch to apply to the changes Linus did recently to convert to > the fttrm010

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Thu, 2017-05-25 at 22:28 +0200, Daniel Lezcano wrote: > On Tue, May 16, 2017 at 03:58:40PM +0800, Andrew Jeffery wrote: > > Also clean up space-before-tab issues in the documentation. > > Andrew, > > I reworked the patch to apply to the changes Linus did recently to convert to > the fttrm010

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 03:58:40PM +0800, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. Andrew, I reworked the patch to apply to the changes Linus did recently to convert to the fttrm010 driver. Please have a look at:

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 03:58:40PM +0800, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. Andrew, I reworked the patch to apply to the changes Linus did recently to convert to the fttrm010 driver. Please have a look at:

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 03:58:40PM +0800, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. > > Signed-off-by: Andrew Jeffery > --- > Documentation/devicetree/bindings/timer/moxa,moxart-timer.txt | 5 +++-- > drivers/clocksource/moxart_timer.c

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 03:58:40PM +0800, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. > > Signed-off-by: Andrew Jeffery > --- > Documentation/devicetree/bindings/timer/moxa,moxart-timer.txt | 5 +++-- > drivers/clocksource/moxart_timer.c

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 3:58 PM, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. > > Signed-off-by: Andrew Jeffery Acked-by: Joel Stanley > --- >

Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

On Tue, May 16, 2017 at 3:58 PM, Andrew Jeffery wrote: > Also clean up space-before-tab issues in the documentation. > > Signed-off-by: Andrew Jeffery Acked-by: Joel Stanley > --- > Documentation/devicetree/bindings/timer/moxa,moxart-timer.txt | 5 +++-- > drivers/clocksource/moxart_timer.c

[PATCH] clocksource: moxart: Add AST2500 compatible string

Also clean up space-before-tab issues in the documentation. Signed-off-by: Andrew Jeffery --- Documentation/devicetree/bindings/timer/moxa,moxart-timer.txt | 5 +++-- drivers/clocksource/moxart_timer.c| 3 ++- 2 files changed, 5 insertions(+), 3

[PATCH] clocksource: moxart: Add AST2500 compatible string

Also clean up space-before-tab issues in the documentation. Signed-off-by: Andrew Jeffery --- Documentation/devicetree/bindings/timer/moxa,moxart-timer.txt | 5 +++-- drivers/clocksource/moxart_timer.c| 3 ++- 2 files changed, 5 insertions(+), 3 deletions(-) diff