[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-12-01 Thread Thomas Monjalon
> > This patch fixes the warnings and error reported by clang compiler on Linux. > > > > Reported-by: Bruce Richardson > > Signed-off-by: Sujith Sankar > > This PMD now compiles up with clang on FreeBSD. The patch seems rather large > though, > are all these changes necessary for clang compila

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-12-01 Thread Bruce Richardson
On Mon, Dec 01, 2014 at 11:06:43AM +, Sujith Sankar (ssujith) wrote: > > On 01/12/14 4:27 pm, "Bruce Richardson" wrote: > > >On Sat, Nov 29, 2014 at 12:47:37PM +0530, Sujith Sankar wrote: > >> This patch fixes the warnings and error reported by clang compiler on > >>Linux. > >> > >> Reporte

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-12-01 Thread Sujith Sankar (ssujith)
On 01/12/14 4:27 pm, "Bruce Richardson" wrote: >On Sat, Nov 29, 2014 at 12:47:37PM +0530, Sujith Sankar wrote: >> This patch fixes the warnings and error reported by clang compiler on >>Linux. >> >> Reported-by: Bruce Richardson >> Signed-off-by: Sujith Sankar > >This PMD now compiles up with

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-12-01 Thread Bruce Richardson
On Sat, Nov 29, 2014 at 12:47:37PM +0530, Sujith Sankar wrote: > This patch fixes the warnings and error reported by clang compiler on Linux. > > Reported-by: Bruce Richardson > Signed-off-by: Sujith Sankar This PMD now compiles up with clang on FreeBSD. The patch seems rather large though, ar

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-11-29 Thread Sujith Sankar (ssujith)
On 29/11/14 4:47 pm, "Thomas Monjalon" wrote: >29/11/2014 07:22, Sujith Sankar : >> Sorry for the typo in the subject. It is enicpmd. > >Actually no, it should be enic. Usually we use enic to denote the kernel mode driver and enicpmd to denote the PMD. But it is alright to follow what?s being

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-11-29 Thread Sujith Sankar
This patch fixes the warnings and error reported by clang compiler on Linux. Reported-by: Bruce Richardson Signed-off-by: Sujith Sankar --- lib/librte_pmd_enic/enic.h | 40 + lib/librte_pmd_enic/enic_compat.h | 1 + lib/librte_pmd_enic/enic_main.c

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-11-29 Thread Thomas Monjalon
29/11/2014 07:22, Sujith Sankar : > Sorry for the typo in the subject. It is enicpmd. Actually no, it should be enic. For virtio, it is not virtiopmd, for ixgbe, it is not ixgbepmd, etc :) Talking about title, please start with lower case and a verb, e.g. fix build with clang. To have a good ide

[dpdk-dev] [PATCH] enicpd: Warnings and one error when built using clang compiler

2014-11-29 Thread Sujith Sankar (ssujith)
Hi all, Sorry for the typo in the subject. It is enicpmd. Bruce, Please try out this patch on FreeBSD and let me know if any fix is required. Thanks, -Sujith On 29/11/14 12:47 pm, "Sujith Sankar (ssujith)" wrote: >This patch fixes the warnings and error reported by clang compiler on >Linux. >