Re: [lng-odp] [PATCH v2] helper: convert to a library

2015-07-09 Thread Maxim Uvarov
Merged, Thanks, Maxim. On 07/06/15 22:13, Mike Holmes wrote: Remove the need to build helper source files into the linux-generic library by converting helpers to be their own library. This removes the need for all other platforms to also build in the helpers which are optional just to run the

Re: [lng-odp] [PATCH v2] helper: convert to a library

2015-07-08 Thread Maxim Uvarov
On 07/07/15 21:20, Mike Holmes wrote: On 7 July 2015 at 06:34, Maxim Uvarov maxim.uva...@linaro.org mailto:maxim.uva...@linaro.org wrote: this path install helper headers to /usr/include that is not correct. Right place is /usr/include/odp/helper due to 2 reasons: 1. be

Re: [lng-odp] [PATCH v2] helper: convert to a library

2015-07-08 Thread Mike Holmes
On 8 July 2015 at 07:28, Maxim Uvarov maxim.uva...@linaro.org wrote: On 07/07/15 21:20, Mike Holmes wrote: On 7 July 2015 at 06:34, Maxim Uvarov maxim.uva...@linaro.org mailto: maxim.uva...@linaro.org wrote: this path install helper headers to /usr/include that is not correct.

Re: [lng-odp] [PATCH v2] helper: convert to a library

2015-07-07 Thread Christophe Milard
On 2015-07-06 15:13, Mike Holmes wrote: Remove the need to build helper source files into the linux-generic library by converting helpers to be their own library. This removes the need for all other platforms to also build in the helpers which are optional just to run the tests.

Re: [lng-odp] [PATCH v2] helper: convert to a library

2015-07-07 Thread Mike Holmes
On 7 July 2015 at 06:34, Maxim Uvarov maxim.uva...@linaro.org wrote: this path install helper headers to /usr/include that is not correct. Right place is /usr/include/odp/helper due to 2 reasons: 1. be consistent to other projects and headers: /usr/include/linux/udp.h

Re: [lng-odp] [PATCH v2] helper: convert to a library

2015-07-07 Thread Maxim Uvarov
this path install helper headers to /usr/include that is not correct. Right place is /usr/include/odp/helper due to 2 reasons: 1. be consistent to other projects and headers: /usr/include/linux/udp.h /usr/include/netinet/udp.h So put it under odp. 2. C code has: odp/helper/ring.h Please do