Message ID | 1486521424-11172-1-git-send-email-kevin.wang@linaro.org |
---|---|
Headers | show |
Series | a new API to support timer capability | expand |
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(-) >
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, 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(-) >> > >> >> >
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(-) >>> > >>> >>> >> >
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