Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-25 Thread Maxim Uvarov
Merged, Maxim. On 01/25/17 18:59, Bill Fischofer wrote: > I also think this cleanup is a step in the right direction. If the > helpers are to be an adjunct ODP package that can be distributed > alongside ODP then this is what's needed. Otherwise they are just a > disjoint set of unrelated

Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-25 Thread Bill Fischofer
I also think this cleanup is a step in the right direction. If the helpers are to be an adjunct ODP package that can be distributed alongside ODP then this is what's needed. Otherwise they are just a disjoint set of unrelated individual files that we should not try to package at all. But that

Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-25 Thread Christophe Milard
For the series: Reviewed-by: Christophe Milard On 23 January 2017 at 20:46, Mike Holmes wrote: > Greatly reduce the proliferation of helper includes that every app needs > Make the public helper API very obvious > Fix recent inclusion of

Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-25 Thread Christophe Milard
ilto:lng-odp-boun...@lists.linaro.org] On Behalf Of Mike >>> Holmes >>> Sent: Monday, January 23, 2017 9:46 PM >>> To: lng-odp@lists.linaro.org >>> Subject: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public >>> helper API >>> >>&g

Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-24 Thread Mike Holmes
uary 23, 2017 9:46 PM >> To: lng-odp@lists.linaro.org >> Subject: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public >> helper API >> >> Greatly reduce the proliferation of helper includes that every app needs >> Make the public helper API very obvious >&

Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-24 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Mike > Holmes > Sent: Monday, January 23, 2017 9:46 PM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public > helper AP

Re: [lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-23 Thread Brian Brooks
Reviewed-and-tested-by: Brian Brooks On 01/23 14:46:04, Mike Holmes wrote: > Greatly reduce the proliferation of helper includes that every app needs > Make the public helper API very obvious > Fix recent inclusion of table APIs that were not in the helper include dir

[lng-odp] [PATCH 0/4] introduce odph_api.h and clean up public helper API

2017-01-23 Thread Mike Holmes
Greatly reduce the proliferation of helper includes that every app needs Make the public helper API very obvious Fix recent inclusion of table APIs that were not in the helper include dir and were not exported during install. Mike Holmes (4): helper: add odph_api.h for existing exported headers