Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1652130 > unrolled thread

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

Started byLinus Walleij <linus.walleij@linaro.org>
First post2017-05-28 16:00 +0200
Last post2017-05-30 12:10 +0200
Articles 6 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] clocksource: moxart: Add AST2500 compatible string Linus Walleij <linus.walleij@linaro.org> - 2017-05-28 16:00 +0200
    Re: [PATCH] clocksource: moxart: Add AST2500 compatible string Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-05-28 20:30 +0200
      Re: [PATCH] clocksource: moxart: Add AST2500 compatible string Linus Walleij <linus.walleij@linaro.org> - 2017-05-30 09:50 +0200
        Re: [PATCH] clocksource: moxart: Add AST2500 compatible string Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-05-30 11:00 +0200
          Re: [PATCH] clocksource: moxart: Add AST2500 compatible string Linus Walleij <linus.walleij@linaro.org> - 2017-05-30 12:00 +0200
            Re: [PATCH] clocksource: moxart: Add AST2500 compatible string Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-05-30 12:10 +0200

#1652130 — Re: [PATCH] clocksource: moxart: Add AST2500 compatible string

FromLinus Walleij <linus.walleij@linaro.org>
Date2017-05-28 16:00 +0200
SubjectRe: [PATCH] clocksource: moxart: Add AST2500 compatible string
Message-ID<tM6TE-2Jg-11@gated-at.bofh.it>
On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery <andrew@aj.id.au> wrote:

> Also clean up space-before-tab issues in the documentation.
>
> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Does this collide with Daniel's 2500 patch?

Sorry for the mess, tell me if I need to fix something up :(

Yours,
Linus Walleij

[toc] | [next] | [standalone]

#1652191

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2017-05-28 20:30 +0200
Message-ID<tMb6V-5wv-5@gated-at.bofh.it>
In reply to#1652130
On 28/05/2017 15:59, Linus Walleij wrote:
> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
> 
>> Also clean up space-before-tab issues in the documentation.
>>
>> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> 
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> 
> Does this collide with Daniel's 2500 patch?
> 
> Sorry for the mess, tell me if I need to fix something up :(

It is ok, an usual collision change to be fixed when merging ;)

By the way, can you have a look at fttmr010_read_sched_clock to remove
the local_fttmr->count_down test?

Thanks.

  -- Daniel

-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

[toc] | [prev] | [next] | [standalone]

#1652908

FromLinus Walleij <linus.walleij@linaro.org>
Date2017-05-30 09:50 +0200
Message-ID<tMK4G-4lV-9@gated-at.bofh.it>
In reply to#1652191
On Sun, May 28, 2017 at 8:27 PM, Daniel Lezcano
<daniel.lezcano@linaro.org> wrote:
> On 28/05/2017 15:59, Linus Walleij wrote:
>> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
>>
>>> Also clean up space-before-tab issues in the documentation.
>>>
>>> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
>>
>> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>>
>> Does this collide with Daniel's 2500 patch?
>>
>> Sorry for the mess, tell me if I need to fix something up :(
>
> It is ok, an usual collision change to be fixed when merging ;)
>
> By the way, can you have a look at fttmr010_read_sched_clock to remove
> the local_fttmr->count_down test?

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.)

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]

#1652966

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2017-05-30 11:00 +0200
Message-ID<tMLar-54b-25@gated-at.bofh.it>
In reply to#1652908
On 30/05/2017 09:44, Linus Walleij wrote:
> On Sun, May 28, 2017 at 8:27 PM, Daniel Lezcano
> <daniel.lezcano@linaro.org> wrote:
>> On 28/05/2017 15:59, Linus Walleij wrote:
>>> On Tue, May 16, 2017 at 9:58 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
>>>
>>>> Also clean up space-before-tab issues in the documentation.
>>>>
>>>> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
>>>
>>> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>>>
>>> Does this collide with Daniel's 2500 patch?
>>>
>>> Sorry for the mess, tell me if I need to fix something up :(
>>
>> It is ok, an usual collision change to be fixed when merging ;)
>>
>> By the way, can you have a look at fttmr010_read_sched_clock to remove
>> the local_fttmr->count_down test?
> 
> 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.)

Yes, thanks Linus!

  -- Daniel



-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

[toc] | [prev] | [next] | [standalone]

#1653060

FromLinus Walleij <linus.walleij@linaro.org>
Date2017-05-30 12:00 +0200
Message-ID<tMM6u-5FJ-19@gated-at.bofh.it>
In reply to#1652966
On Tue, May 30, 2017 at 10:54 AM, Daniel Lezcano
<daniel.lezcano@linaro.org> 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.)
>
> Yes, thanks Linus!

Can you point me to a git tree with all your patches so I can work
on top of what you have?

I think I will also look into adding delay timers.

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]

#1653065

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2017-05-30 12:10 +0200
Message-ID<tMMg9-5Z0-7@gated-at.bofh.it>
In reply to#1653060
On Tue, May 30, 2017 at 11:53:35AM +0200, Linus Walleij wrote:
> On Tue, May 30, 2017 at 10:54 AM, Daniel Lezcano
> <daniel.lezcano@linaro.org> 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.)
> >
> > Yes, thanks Linus!
> 
> Can you point me to a git tree with all your patches so I can work
> on top of what you have?
> 
> I think I will also look into adding delay timers.

https://git.linaro.org/people/daniel.lezcano/linux.git clockevents/4.13

(subject to rebase)

-- 

 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web