On Tue, Oct 05, 2010 at 12:43:47PM -0400, jason wrote: > Greg, > > Greg KH wrote: > >On Thu, Sep 30, 2010 at 03:15:32PM -0400, Jason Cooper wrote: > >>Jason Cooper (9): > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch errors 'assignment in if > >> condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > >> staging: brcm80211: fix checkpatch error 'assignment in if condition' > > > >In the future, please use a unique "Subject" line, not all the same > >thing, as it really isn't that descriptive in the end :) > > > > No problem, the only differences between them were they each operated > on a separate file. In the future, would you want them all grouped as > one? I didn't do that this time because it would have been one large > patch and, I thought, difficult to review. Just list the filename as part of the subject: staging: brcm80211: foo.c: fix checkpatch error 'assignment in if condition' That makes it unique and a bit more descriptive. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
