Re: [dpdk-dev] [PATCH v2 2/4] Driver/Mellanox: fix PMD compiling issue

2018-05-15 Thread Sirshak Das
Hi Yongseok, We will make the commit message more verbose. As for the error: This is a clang compiler issue: drivers/net/mlx5/mlx5_rxtx_vec.c:37: /home/sirdas/code/commitc/dpdk-stable-18.02.1/drivers/net/mlx5/mlx5_rxtx_vec_neon.h:170:24: error: too many arguments provided to function-like macro i

Re: [dpdk-dev] [PATCH v2 2/4] Driver/Mellanox: fix PMD compiling issue

2018-05-15 Thread Yongseok Koh
> On May 15, 2018, at 10:52 AM, Sirshak Das wrote: > > Hi Yongseok, > > We will make the commit message more verbose. > As for the error: This is a clang compiler issue: > drivers/net/mlx5/mlx5_rxtx_vec.c:37: > /home/sirdas/code/commitc/dpdk-stable-18.02.1/drivers/net/mlx5/mlx5_rxtx_vec_neon.h:

Re: [dpdk-dev] [PATCH v2 2/4] Driver/Mellanox: fix PMD compiling issue

2018-05-15 Thread Yongseok Koh
> On May 15, 2018, at 2:23 AM, Gavin Hu wrote: The title should start from 'net/mlx5: ...' and please be more specific why that pair of parentheses is needed. Add more commit messages and it'd be better to add the error messages like you do in the other commit. To me, it still looks unnecessary

[dpdk-dev] [PATCH v2 2/4] Driver/Mellanox: fix PMD compiling issue

2018-05-15 Thread Gavin Hu
Signed-off-by: Gavin Hu Signed-off-by: Sirshak Das Reviewed-by: Phil Yang Reviewed-by: Honnappa Nagarahalli --- drivers/net/mlx5/mlx5_rxtx_vec_neon.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx_vec_neon.h b/drivers/net/mlx5/mlx5_