> > 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
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.
> >>
> >>
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
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
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 +
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
6 matches
Mail list logo