linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/8] patch series
@ 2014-04-19  0:13 Ryan Desfosses
  0 siblings, 0 replies; 3+ messages in thread
From: Ryan Desfosses @ 2014-04-19  0:13 UTC (permalink / raw)
  To: bhelgaas; +Cc: trivial, linux-kernel, linux-pci

There are eight patches in this series.  All changes are trivial with no
dependencies or required order.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 0/8] patch series
@ 2014-04-19  0:16 Ryan Desfosses
  2014-04-19  0:25 ` Randy Dunlap
  0 siblings, 1 reply; 3+ messages in thread
From: Ryan Desfosses @ 2014-04-19  0:16 UTC (permalink / raw)
  To: bhelgaas; +Cc: trivial, linux-kernel, linux-pci

There are eight patches in this series.  All changes are trivial with no
dependencies or required order.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/8] patch series
  2014-04-19  0:16 [PATCH 0/8] patch series Ryan Desfosses
@ 2014-04-19  0:25 ` Randy Dunlap
  0 siblings, 0 replies; 3+ messages in thread
From: Randy Dunlap @ 2014-04-19  0:25 UTC (permalink / raw)
  To: Ryan Desfosses, bhelgaas; +Cc: trivial, linux-kernel, linux-pci

On 04/18/2014 05:16 PM, Ryan Desfosses wrote:
> There are eight patches in this series.  All changes are trivial with no
> dependencies or required order.
> --

My small comments:

a. The Subject should have mentioned PCI and trivial.

b. Please use imperative mood for patch descriptions.  I.e.,

use remove instead of removed;
use move instead of moved;
use merge instead of merged;
use add instead of added.


Don't resend the patches based only on my comments (unless the PCI
maintainer asks you to do so).


thanks,
-- 
~Randy

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-04-19  0:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-19  0:16 [PATCH 0/8] patch series Ryan Desfosses
2014-04-19  0:25 ` Randy Dunlap
  -- strict thread matches above, loose matches on Subject: below --
2014-04-19  0:13 Ryan Desfosses

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).