Messages in this thread | | | Date | Fri, 18 Apr 2014 17:25:53 -0700 | From | Randy Dunlap <> | Subject | Re: [PATCH 0/8] patch series |
| |
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
| |