[Intel-wired-lan] [PATCH v2] fm10k: don't initialize fm10k_workqueue at global level
Keller, Jacob E
jacob.e.keller at intel.com
Tue Nov 3 23:19:40 UTC 2015
On Tue, 2015-11-03 at 23:15 +0000, Allan, Bruce W wrote:
> > -----Original Message-----
> > From: Keller, Jacob E
> > Sent: Tuesday, November 03, 2015 3:05 PM
> > To: Allan, Bruce W; intel-wired-lan at lists.osuosl.org
> > Subject: Re: [PATCH v2] fm10k: don't initialize fm10k_workqueue at
> > global
> > level
> >
> > On Tue, 2015-11-03 at 22:40 +0000, Allan, Bruce W wrote:
> > > > -----Original Message-----
> > > > From: Keller, Jacob E
> > > > Sent: Tuesday, November 03, 2015 11:35 AM
> > > > To: Intel Wired LAN
> > > > Cc: Allan, Bruce W; Keller, Jacob E
> > > > Subject: [PATCH v2] fm10k: don't initialize fm10k_workqueue at
> > > > global level
> > > >
> > > > From: Bruce Allan <bruce.w.allan at intel.com>
> > > >
> > > > Cleans up checkpatch GLOBAL_INITIALIZERS error
> > > >
> > > > Signed-off-by: Bruce Allan <bruce.w.allan at intel.com>
> > > > Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> > > > ---
> > > > -v2
> > > > * Fix title from checkpatch complaint
> > >
> > > Out of curiosity, what was wrong with the original v1 patch title
> > > and
> > > what was the exact checkpatch complaint?
> > >
> >
> > checkpatch.pl results for patch "[PATCH] fm10k: fix checkpatch
> > GLOBAL_INITIALISERS error"
> >
> > WARNING:EMAIL_SUBJECT: A patch subject line should describe the
> > change
> > not the tool that found it
> > #4:
> > Subject: [PATCH] fm10k: fix checkpatch GLOBAL_INITIALISERS error
> >
> > total: 0 errors, 1 warnings, 0 checks, 24 lines checked
> >
> >
> > Regards,
> > Jake
>
> Which version of checkpatch and options did you use? I tried with
> the one in Jeff's tree and others but do not get the same.
The one in Jeff's tree did it for me, and has been since commit
a2fe16b9d878 ("checkpatch: try to avoid poor patch subject lines")
I get:
WARNING: A patch subject line should describe the change not the tool that found it
#4:
Subject: [PATCH] fm10k: fix checkpatch GLOBAL_INITIALISERS error
total: 0 errors, 1 warnings, 0 checks, 24 lines checked
Your patch has style problems, please review.
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
Regards,
Jake
More information about the Intel-wired-lan
mailing list