[dpdk-dev] checkpatch script
De Lara Guarch, Pablo
pablo.de.lara.guarch at intel.com
Tue Nov 25 15:48:49 CET 2014
> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon
> Sent: Tuesday, November 25, 2014 2:29 PM
> To: De Lara Guarch, Pablo
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] checkpatch script
>
> Hi Pablo,
>
> 2014-11-25 13:11, De Lara Guarch, Pablo:
> > Hi Thomas,
> >
> > As we have seen the issues we have had on previous patches due to
> > checkpatch,
>
> No we have no issues because of checkpatch ;)
> It helps us to see some obvious errors.
I agree. With issues, I meant just confusion between using one version of checkpatch or another
(mind that you are using script from 3.18-rc6, the very latest one), so it was good that you mentioned the version before :)
>
> > would it be possible to include the up-to-date script in the
> > repo? I have realized that people either do not check their patches or check
> > them with an old version that do not catch some errors that the latest one
> > does, so I think this is a problem.
>
> The real problem is to clearly define all the coding rules and adapt
> checkpatch to our needs.
Agree 100%.
>
> > Plus, I think it would be a good idea to mention it in dpdk.org/dev, as this
> > is clearly a requirement for merging patches.
>
> Please, let's close DPDK 1.8 first.
> Then we'll have to write some documents and tools to make them approved
> after necessary discussions/debates.
OK. We will discuss it for 2.0. Thanks Thomas!
>
> --
> Thomas
More information about the dev
mailing list