Re: [lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-08 Thread Savolainen, Petri (NSN - FI/Espoo)
Kondratiuk Cc: lng-odp; Savolainen, Petri (NSN - FI/Espoo) Subject: Re: [lng-odp] [PATCH] api: odp_align: move internal macros I addressed only what was in the 1.0 API dochttps://docs.google.com/a/linaro.org/document/d/1BRVyW8IIVMTq4nhB_vUz5y-te6TEdu5g1XgolujjY6c/edit#heading=h.iwjfgdfj3au0 we need

Re: [lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-05 Thread Taras Kondratiuk
On 12/04/2014 10:25 PM, Mike Holmes wrote: @@ -138,12 +114,6 @@ extern C { #define ODP_ALIGN_ROUNDDOWN_PTR_POWER_2(x, align)\ ((void *)ODP_ALIGN_ROUNDDOWN_POWER_2((uintptr_t)(x), (uintptr_t)(align))) Only part of internal macros were moved. For example why ODP_ALIGN_ROUNDDOWN_PTR_POWER_2()

Re: [lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-05 Thread Ola Liljedahl
On 5 December 2014 at 10:04, Taras Kondratiuk taras.kondrat...@linaro.org wrote: On 12/04/2014 10:25 PM, Mike Holmes wrote: @@ -138,12 +114,6 @@ extern C { #define ODP_ALIGN_ROUNDDOWN_PTR_POWER_2(x, align)\ ((void *)ODP_ALIGN_ROUNDDOWN_POWER_2((uintptr_t)(x), (uintptr_t)(align))) Only

[lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-04 Thread Mike Holmes
Signed-off-by: Mike Holmes mike.hol...@linaro.org --- platform/linux-generic/include/api/odp_align.h | 50 --- .../linux-generic/include/odp_align_internal.h | 100 + .../linux-generic/include/odp_packet_io_internal.h | 1 +

Re: [lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-04 Thread Bill Fischofer
Looks good. Please get this merged soon as it will cause a number of other patches to need to be rebased, including my buffer pool stuff that I'm reworking for resubmission. Thanks. On Thu, Dec 4, 2014 at 2:25 PM, Mike Holmes mike.hol...@linaro.org wrote: Signed-off-by: Mike Holmes

Re: [lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-04 Thread Maxim Uvarov
Merged, Maxim. On 12/05/2014 01:11 AM, Bill Fischofer wrote: Looks good. Please get this merged soon as it will cause a number of other patches to need to be rebased, including my buffer pool stuff that I'm reworking for resubmission. Thanks. On Thu, Dec 4, 2014 at 2:25 PM, Mike Holmes