Re: [dpdk-dev] checkpatch.pl inconsistent results

2017-02-26 Thread Legacy, Allain
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Sunday, February 26, 2017 11:42 AM > To: Legacy, Allain > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] checkpatch.pl inconsistent results ... > Yes it has been discussed few tim

Re: [dpdk-dev] checkpatch.pl inconsistent results

2017-02-26 Thread Thomas Monjalon
2017-02-26 16:20, Legacy, Allain: > From: Thomas Monjalon > > It is a false positive. > > PRIx64 and PRIu64 are obviously allowed. > > The only thing you need to take care is having spaces around. > > > Ok, thanks Thomas. Obviously PRIx64 and PRIu64 are acceptable so I knew that > something must

Re: [dpdk-dev] checkpatch.pl inconsistent results

2017-02-26 Thread Legacy, Allain
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Sunday, February 26, 2017 4:12 AM > To: Legacy, Allain > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] checkpatch.pl inconsistent results > Importance: High > > 2017-

Re: [dpdk-dev] checkpatch.pl inconsistent results

2017-02-26 Thread Thomas Monjalon
2017-02-25 11:54, Legacy, Allain: > Hi, > I sent a patchset to the to the mailing list last night for which I received > several coding style warnings. Having discovered that I was using an older > version of checkpatch.pl I downloaded the latest and set out to fix the > warnings. The tool is

[dpdk-dev] checkpatch.pl inconsistent results

2017-02-25 Thread Legacy, Allain
Hi, I sent a patchset to the to the mailing list last night for which I received several coding style warnings. Having discovered that I was using an older version of checkpatch.pl I downloaded the latest and set out to fix the warnings. The tool is flagging the usage of PRIx64 and PRIu64 in