* Re: [drm-intel:drm-intel-fixes 52/52] ERROR: Unrecognized email address: 'stable.]' [not found] ` <20130717091028.GA31187@localhost> @ 2013-07-17 9:29 ` Daniel Vetter 2013-07-17 10:01 ` Joe Perches 0 siblings, 1 reply; 4+ messages in thread From: Daniel Vetter @ 2013-07-17 9:29 UTC (permalink / raw) To: Fengguang Wu Cc: Konstantin Khlebnikov, kbuild-all, Linux Kernel Mailing List, Joe Perches, intel-gfx On Wed, Jul 17, 2013 at 11:10 AM, Fengguang Wu <fengguang.wu@intel.com> wrote: > > Hi Konstantin, > > FYI, there are new warnings show up in > > tree: git://people.freedesktop.org/~danvet/drm-intel.git drm-intel-fixes > head: 71e4092e52499ec74bc1dec0f883b15f2c424ec5 > commit: 71e4092e52499ec74bc1dec0f883b15f2c424ec5 [52/52] drm/i915: fix long-standing SNB regression in power consumption after resume v2 > > scripts/checkpatch.pl 0001-drm-i915-fix-long-standing-SNB-regression-in-power-c.patch > > ERROR: Unrecognized email address: 'stable.]' > #41: > cc: stable.] Well, I've added that while applying the patch - I tend to smash maintainer notes into the sob section and word-wraping caused the cc: stable remark to be parsed. Is there an officially sanctioned way for such notes that appeases checkpatch? Adding lkml and checkpatch maintainer. -Daniel > --- > 0-DAY kernel build testing backend Open Source Technology Center > http://lists.01.org/mailman/listinfo/kbuild Intel Corporation -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [drm-intel:drm-intel-fixes 52/52] ERROR: Unrecognized email address: 'stable.]' 2013-07-17 9:29 ` [drm-intel:drm-intel-fixes 52/52] ERROR: Unrecognized email address: 'stable.]' Daniel Vetter @ 2013-07-17 10:01 ` Joe Perches 2013-07-17 11:24 ` Daniel Vetter 2013-07-18 14:58 ` Fengguang Wu 0 siblings, 2 replies; 4+ messages in thread From: Joe Perches @ 2013-07-17 10:01 UTC (permalink / raw) To: Daniel Vetter Cc: Fengguang Wu, Konstantin Khlebnikov, kbuild-all, Linux Kernel Mailing List, intel-gfx On Wed, 2013-07-17 at 11:29 +0200, Daniel Vetter wrote: > On Wed, Jul 17, 2013 at 11:10 AM, Fengguang Wu <fengguang.wu@intel.com> wrote: > > FYI, there are new warnings show up in > > tree: git://people.freedesktop.org/~danvet/drm-intel.git drm-intel-fixes > > head: 71e4092e52499ec74bc1dec0f883b15f2c424ec5 > > commit: 71e4092e52499ec74bc1dec0f883b15f2c424ec5 [52/52] drm/i915: fix long-standing SNB regression in power consumption after resume v2 > > > > scripts/checkpatch.pl 0001-drm-i915-fix-long-standing-SNB-regression-in-power-c.patch > > > > ERROR: Unrecognized email address: 'stable.]' > > #41: > > cc: stable.] > > Well, I've added that while applying the patch - I tend to smash > maintainer notes into the sob section and word-wraping caused the cc: > stable remark to be parsed. Is there an officially sanctioned way for > such notes that appeases checkpatch? Adding lkml and checkpatch > maintainer. > -Daniel (It would have been nice to get the content that failes instead of having to pull the tree) Don't wrap text to start a line with cc: Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Jesse Barnes <jbarnes@virtuousgeek.org> [danvet: Add note about v1 vs. v2 of this patch and use standard layout for the commit citation. Also add the tested-bys from v1 and a cc: stable.] Cc: stable@vger.kernel.org (Note: tiny conflict due to the addition of You could have added something like: [danvet: Add note about v1 vs. v2 of this patch and use standard layout for the commit citation. Also add the tested-bys from v1 and a cc: stable.] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [drm-intel:drm-intel-fixes 52/52] ERROR: Unrecognized email address: 'stable.]' 2013-07-17 10:01 ` Joe Perches @ 2013-07-17 11:24 ` Daniel Vetter 2013-07-18 14:58 ` Fengguang Wu 1 sibling, 0 replies; 4+ messages in thread From: Daniel Vetter @ 2013-07-17 11:24 UTC (permalink / raw) To: Joe Perches Cc: Daniel Vetter, Fengguang Wu, Konstantin Khlebnikov, kbuild-all, Linux Kernel Mailing List, intel-gfx On Wed, Jul 17, 2013 at 03:01:38AM -0700, Joe Perches wrote: > On Wed, 2013-07-17 at 11:29 +0200, Daniel Vetter wrote: > > On Wed, Jul 17, 2013 at 11:10 AM, Fengguang Wu <fengguang.wu@intel.com> wrote: > > > FYI, there are new warnings show up in > > > tree: git://people.freedesktop.org/~danvet/drm-intel.git drm-intel-fixes > > > head: 71e4092e52499ec74bc1dec0f883b15f2c424ec5 > > > commit: 71e4092e52499ec74bc1dec0f883b15f2c424ec5 [52/52] drm/i915: fix long-standing SNB regression in power consumption after resume v2 > > > > > > scripts/checkpatch.pl 0001-drm-i915-fix-long-standing-SNB-regression-in-power-c.patch > > > > > > ERROR: Unrecognized email address: 'stable.]' > > > #41: > > > cc: stable.] > > > > Well, I've added that while applying the patch - I tend to smash > > maintainer notes into the sob section and word-wraping caused the cc: > > stable remark to be parsed. Is there an officially sanctioned way for > > such notes that appeases checkpatch? Adding lkml and checkpatch > > maintainer. > > -Daniel > > (It would have been nice to get the content that failes > instead of having to pull the tree) Oops, sorry. > Don't wrap text to start a line with cc: > > Cc: Chris Wilson <chris@chris-wilson.co.uk> > Cc: Jesse Barnes <jbarnes@virtuousgeek.org> > [danvet: Add note about v1 vs. v2 of this patch and use standard > layout for the commit citation. Also add the tested-bys from v1 and a > cc: stable.] > Cc: stable@vger.kernel.org (Note: tiny conflict due to the addition of > > You could have added something like: > > [danvet: Add note about v1 vs. v2 of this patch and use standard > layout for the commit citation. > Also add the tested-bys from v1 and a cc: stable.] Ok, I'll try to rewrap if the cc: hits the first row next time around - just didn't occur to me that checkpatch would notice and I tend to not re-run checkpatch before pushing when editing a patch a bit. Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [drm-intel:drm-intel-fixes 52/52] ERROR: Unrecognized email address: 'stable.]' 2013-07-17 10:01 ` Joe Perches 2013-07-17 11:24 ` Daniel Vetter @ 2013-07-18 14:58 ` Fengguang Wu 1 sibling, 0 replies; 4+ messages in thread From: Fengguang Wu @ 2013-07-18 14:58 UTC (permalink / raw) To: Joe Perches Cc: Daniel Vetter, Konstantin Khlebnikov, kbuild-all, Linux Kernel Mailing List, intel-gfx Joe, > (It would have been nice to get the content that failes > instead of having to pull the tree) Good suggestion! I'll attach the git-format-patch result in the checkpatch.pl reports in future. Thanks, Fengguang ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-07-18 14:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <51e63ed0.zKgIdwWoDkYbRWqH%fengguang.wu@intel.com>
[not found] ` <20130717091028.GA31187@localhost>
2013-07-17 9:29 ` [drm-intel:drm-intel-fixes 52/52] ERROR: Unrecognized email address: 'stable.]' Daniel Vetter
2013-07-17 10:01 ` Joe Perches
2013-07-17 11:24 ` Daniel Vetter
2013-07-18 14:58 ` Fengguang Wu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox