mbox series

[API-NEXT,PATCHv4,0/4] a new API to support timer capability

Message ID 1486521424-11172-1-git-send-email-kevin.wang@linaro.org
Headers show
Series a new API to support timer capability | expand

Message

Kevin Wang Feb. 8, 2017, 2:37 a.m. UTC
*** BLURB HERE ***

Kevin Wang (4):
  api: timer: add odp_timer_capability() api
  linux-generic: timer: implement odp_timer_capability()
  validation/example: call odp_timer_capability() before creating a
    timer pool
  doc: userguide: add odp_timer_capability() section to user guide

 doc/users-guide/users-guide-timer.adoc        |  3 +-
 example/generator/odp_generator.c             |  8 ++-
 example/timer/odp_timer_simple.c              |  9 ++-
 example/timer/odp_timer_test.c                |  7 +-
 include/odp/api/spec/timer.h                  | 21 ++++++
 platform/linux-generic/odp_timer.c            | 95 +++++++++++++++++++++++++++
 test/common_plat/validation/api/timer/timer.c | 28 +++++---
 7 files changed, 158 insertions(+), 13 deletions(-)

-- 
1.9.1

Comments

Maxim Uvarov Feb. 16, 2017, 6:59 p.m. UTC | #1
Petri, please review API patch.

Maxim.


On 02/08/17 05:37, Kevin Wang wrote:
> *** BLURB HERE ***

> 

> Kevin Wang (4):

>   api: timer: add odp_timer_capability() api

>   linux-generic: timer: implement odp_timer_capability()

>   validation/example: call odp_timer_capability() before creating a

>     timer pool

>   doc: userguide: add odp_timer_capability() section to user guide

> 

>  doc/users-guide/users-guide-timer.adoc        |  3 +-

>  example/generator/odp_generator.c             |  8 ++-

>  example/timer/odp_timer_simple.c              |  9 ++-

>  example/timer/odp_timer_test.c                |  7 +-

>  include/odp/api/spec/timer.h                  | 21 ++++++

>  platform/linux-generic/odp_timer.c            | 95 +++++++++++++++++++++++++++

>  test/common_plat/validation/api/timer/timer.c | 28 +++++---

>  7 files changed, 158 insertions(+), 13 deletions(-)

>
Yi He July 11, 2017, 2:58 a.m. UTC | #2
Hi, Maxim Petri and team,

A ping for ODP-502, the patch series reviewed but not merged eventually,
just like to know how to progress on this ticket?

ODP-502 Capabilities structs for timer & timers
http://patches.opendataplane.org/patch/8078/

Thanks and best regards, Yi


On 17 February 2017 at 02:59, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> Petri, please review API patch.

>

> Maxim.

>

>

> On 02/08/17 05:37, Kevin Wang wrote:

> > *** BLURB HERE ***

> >

> > Kevin Wang (4):

> >   api: timer: add odp_timer_capability() api

> >   linux-generic: timer: implement odp_timer_capability()

> >   validation/example: call odp_timer_capability() before creating a

> >     timer pool

> >   doc: userguide: add odp_timer_capability() section to user guide

> >

> >  doc/users-guide/users-guide-timer.adoc        |  3 +-

> >  example/generator/odp_generator.c             |  8 ++-

> >  example/timer/odp_timer_simple.c              |  9 ++-

> >  example/timer/odp_timer_test.c                |  7 +-

> >  include/odp/api/spec/timer.h                  | 21 ++++++

> >  platform/linux-generic/odp_timer.c            | 95

> +++++++++++++++++++++++++++

> >  test/common_plat/validation/api/timer/timer.c | 28 +++++---

> >  7 files changed, 158 insertions(+), 13 deletions(-)

> >

>

>
Maxim Uvarov July 11, 2017, 9:53 a.m. UTC | #3
Yi, I need Perti review to merge api-next patch.
Patch looks reasonable for me. Might be to expend it with res_ns_min,
res_ns_max, min_tmo, max_tmo. I.e. define ranges which values are
acceptable fro pool creation.

Maxim.


On 11 July 2017 at 05:58, Yi He <yi.he@linaro.org> wrote:

> Hi, Maxim Petri and team,

>

> A ping for ODP-502, the patch series reviewed but not merged eventually,

> just like to know how to progress on this ticket?

>

> ODP-502 Capabilities structs for timer & timers

> http://patches.opendataplane.org/patch/8078/

>

> Thanks and best regards, Yi

>

>

> On 17 February 2017 at 02:59, Maxim Uvarov <maxim.uvarov@linaro.org>

> wrote:

>

>> Petri, please review API patch.

>>

>> Maxim.

>>

>>

>> On 02/08/17 05:37, Kevin Wang wrote:

>> > *** BLURB HERE ***

>> >

>> > Kevin Wang (4):

>> >   api: timer: add odp_timer_capability() api

>> >   linux-generic: timer: implement odp_timer_capability()

>> >   validation/example: call odp_timer_capability() before creating a

>> >     timer pool

>> >   doc: userguide: add odp_timer_capability() section to user guide

>> >

>> >  doc/users-guide/users-guide-timer.adoc        |  3 +-

>> >  example/generator/odp_generator.c             |  8 ++-

>> >  example/timer/odp_timer_simple.c              |  9 ++-

>> >  example/timer/odp_timer_test.c                |  7 +-

>> >  include/odp/api/spec/timer.h                  | 21 ++++++

>> >  platform/linux-generic/odp_timer.c            | 95

>> +++++++++++++++++++++++++++

>> >  test/common_plat/validation/api/timer/timer.c | 28 +++++---

>> >  7 files changed, 158 insertions(+), 13 deletions(-)

>> >

>>

>>

>
Yi He July 11, 2017, 10 a.m. UTC | #4
Yes, thanks, Maxim, and Petri please wait for Kevin to rebase and work out
a new patch series.

Best Regards, Yi

On 11 July 2017 at 17:53, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> Yi, I need Perti review to merge api-next patch.

> Patch looks reasonable for me. Might be to expend it with res_ns_min,

> res_ns_max, min_tmo, max_tmo. I.e. define ranges which values are

> acceptable fro pool creation.

>

> Maxim.

>

>

> On 11 July 2017 at 05:58, Yi He <yi.he@linaro.org> wrote:

>

>> Hi, Maxim Petri and team,

>>

>> A ping for ODP-502, the patch series reviewed but not merged eventually,

>> just like to know how to progress on this ticket?

>>

>> ODP-502 Capabilities structs for timer & timers

>> http://patches.opendataplane.org/patch/8078/

>>

>> Thanks and best regards, Yi

>>

>>

>> On 17 February 2017 at 02:59, Maxim Uvarov <maxim.uvarov@linaro.org>

>> wrote:

>>

>>> Petri, please review API patch.

>>>

>>> Maxim.

>>>

>>>

>>> On 02/08/17 05:37, Kevin Wang wrote:

>>> > *** BLURB HERE ***

>>> >

>>> > Kevin Wang (4):

>>> >   api: timer: add odp_timer_capability() api

>>> >   linux-generic: timer: implement odp_timer_capability()

>>> >   validation/example: call odp_timer_capability() before creating a

>>> >     timer pool

>>> >   doc: userguide: add odp_timer_capability() section to user guide

>>> >

>>> >  doc/users-guide/users-guide-timer.adoc        |  3 +-

>>> >  example/generator/odp_generator.c             |  8 ++-

>>> >  example/timer/odp_timer_simple.c              |  9 ++-

>>> >  example/timer/odp_timer_test.c                |  7 +-

>>> >  include/odp/api/spec/timer.h                  | 21 ++++++

>>> >  platform/linux-generic/odp_timer.c            | 95

>>> +++++++++++++++++++++++++++

>>> >  test/common_plat/validation/api/timer/timer.c | 28 +++++---

>>> >  7 files changed, 158 insertions(+), 13 deletions(-)

>>> >

>>>

>>>

>>

>
Kevin Wang July 12, 2017, 8:44 a.m. UTC | #5
Patch is rebased. Please help to review.

Thanks
Kevin

2017-07-11 18:00 GMT+08:00 Yi He <yi.he@linaro.org>:

> Yes, thanks, Maxim, and Petri please wait for Kevin to rebase and work out

> a new patch series.

>

> Best Regards, Yi

>

> On 11 July 2017 at 17:53, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

>

> > Yi, I need Perti review to merge api-next patch.

> > Patch looks reasonable for me. Might be to expend it with res_ns_min,

> > res_ns_max, min_tmo, max_tmo. I.e. define ranges which values are

> > acceptable fro pool creation.

> >

> > Maxim.

> >

> >

> > On 11 July 2017 at 05:58, Yi He <yi.he@linaro.org> wrote:

> >

> >> Hi, Maxim Petri and team,

> >>

> >> A ping for ODP-502, the patch series reviewed but not merged eventually,

> >> just like to know how to progress on this ticket?

> >>

> >> ODP-502 Capabilities structs for timer & timers

> >> http://patches.opendataplane.org/patch/8078/

> >>

> >> Thanks and best regards, Yi

> >>

> >>

> >> On 17 February 2017 at 02:59, Maxim Uvarov <maxim.uvarov@linaro.org>

> >> wrote:

> >>

> >>> Petri, please review API patch.

> >>>

> >>> Maxim.

> >>>

> >>>

> >>> On 02/08/17 05:37, Kevin Wang wrote:

> >>> > *** BLURB HERE ***

> >>> >

> >>> > Kevin Wang (4):

> >>> >   api: timer: add odp_timer_capability() api

> >>> >   linux-generic: timer: implement odp_timer_capability()

> >>> >   validation/example: call odp_timer_capability() before creating a

> >>> >     timer pool

> >>> >   doc: userguide: add odp_timer_capability() section to user guide

> >>> >

> >>> >  doc/users-guide/users-guide-timer.adoc        |  3 +-

> >>> >  example/generator/odp_generator.c             |  8 ++-

> >>> >  example/timer/odp_timer_simple.c              |  9 ++-

> >>> >  example/timer/odp_timer_test.c                |  7 +-

> >>> >  include/odp/api/spec/timer.h                  | 21 ++++++

> >>> >  platform/linux-generic/odp_timer.c            | 95

> >>> +++++++++++++++++++++++++++

> >>> >  test/common_plat/validation/api/timer/timer.c | 28 +++++---

> >>> >  7 files changed, 158 insertions(+), 13 deletions(-)

> >>> >

> >>>

> >>>

> >>

> >

>




-- 
Thanks,
Kevin