mbox series

[RFC,0/2] clocksource: davinci-timer: new driver

Message ID 20190417144709.19588-1-brgl@bgdev.pl (mailing list archive)
Headers show
Series clocksource: davinci-timer: new driver | expand

Message

Bartosz Golaszewski April 17, 2019, 2:47 p.m. UTC
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>

Hi Daniel,

as discussed - this is the davinci timer driver split into the clockevent
and clocksource parts.

Since it won't work without all the other (left out for now) changes, I'm
marking it as RFC.

The code has been simplified as requested, the duplicated enums and the
davinci_timer structure have been removed.

Please let me know if that's better. I retested it on da850-lcdk, da830-evm
and dm365-evm.

Bartosz Golaszewski (2):
  clocksource: davinci-timer: add support for clockevents
  clocksource: timer-davinci: add support for clocksource

 drivers/clocksource/Kconfig         |   5 +
 drivers/clocksource/Makefile        |   1 +
 drivers/clocksource/timer-davinci.c | 342 ++++++++++++++++++++++++++++
 include/clocksource/timer-davinci.h |  44 ++++
 4 files changed, 392 insertions(+)
 create mode 100644 drivers/clocksource/timer-davinci.c
 create mode 100644 include/clocksource/timer-davinci.h

Comments

Bartosz Golaszewski May 13, 2019, 7:54 a.m. UTC | #1
śr., 17 kwi 2019 o 16:47 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
>
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> Hi Daniel,
>
> as discussed - this is the davinci timer driver split into the clockevent
> and clocksource parts.
>
> Since it won't work without all the other (left out for now) changes, I'm
> marking it as RFC.
>
> The code has been simplified as requested, the duplicated enums and the
> davinci_timer structure have been removed.
>
> Please let me know if that's better. I retested it on da850-lcdk, da830-evm
> and dm365-evm.
>
> Bartosz Golaszewski (2):
>   clocksource: davinci-timer: add support for clockevents
>   clocksource: timer-davinci: add support for clocksource
>
>  drivers/clocksource/Kconfig         |   5 +
>  drivers/clocksource/Makefile        |   1 +
>  drivers/clocksource/timer-davinci.c | 342 ++++++++++++++++++++++++++++
>  include/clocksource/timer-davinci.h |  44 ++++
>  4 files changed, 392 insertions(+)
>  create mode 100644 drivers/clocksource/timer-davinci.c
>  create mode 100644 include/clocksource/timer-davinci.h
>
> --
> 2.21.0
>

Hi Daniel,

it's been almost a month so a gentle ping. Any comments on that?

Best regards,
Bartosz Golaszewski
Daniel Lezcano May 13, 2019, 8:04 a.m. UTC | #2
On 13/05/2019 09:54, Bartosz Golaszewski wrote:
> śr., 17 kwi 2019 o 16:47 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
>>
>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> Hi Daniel,
>>
>> as discussed - this is the davinci timer driver split into the clockevent
>> and clocksource parts.
>>
>> Since it won't work without all the other (left out for now) changes, I'm
>> marking it as RFC.
>>
>> The code has been simplified as requested, the duplicated enums and the
>> davinci_timer structure have been removed.
>>
>> Please let me know if that's better. I retested it on da850-lcdk, da830-evm
>> and dm365-evm.
>>
>> Bartosz Golaszewski (2):
>>   clocksource: davinci-timer: add support for clockevents
>>   clocksource: timer-davinci: add support for clocksource
>>
>>  drivers/clocksource/Kconfig         |   5 +
>>  drivers/clocksource/Makefile        |   1 +
>>  drivers/clocksource/timer-davinci.c | 342 ++++++++++++++++++++++++++++
>>  include/clocksource/timer-davinci.h |  44 ++++
>>  4 files changed, 392 insertions(+)
>>  create mode 100644 drivers/clocksource/timer-davinci.c
>>  create mode 100644 include/clocksource/timer-davinci.h
>>
>> --
>> 2.21.0
>>
> 
> Hi Daniel,
> 
> it's been almost a month so a gentle ping. Any comments on that?

Oh right, I've been distracted with other things, sorry for that. Let me
review it today.