Re: [PATCH 0/3] fix issues in prism2usb.c.
On 26/05/16 13:32, Kevin wrote: > On Thu, May 26, 2016 at 01:24:28PM +0100, Luis de Bethencourt wrote: >> On 26/05/16 04:22, Kevin McKinney wrote: >>> This patchset will address issues reported by checkpatch.pl tool >>> for file prism2usb.c. >>> >>> Kevin McKinney (3): >>> Staging: wlan-ng: fix alignment not matching. >>> Staging: wlan-ng: fix unnecessary parentheses. >>> Staging: wlan-ng: fix comparison to NULL issue. >>> >>> drivers/staging/wlan-ng/prism2usb.c | 14 +++--- >>> 1 file changed, 7 insertions(+), 7 deletions(-) >>> >> >> Hello Kevin, >> >> Thanks for fixing the issue. >> >> A note for the future, when you resend a patch series you should >> mark it as a new version. So the system maintainer knows the evolution >> of the patchset and to drop the earlier versions in his process. >> >> Look at this random example I just found: >> https://lkml.org/lkml/2016/5/26/161 >> >> If you are using git format-patch, this is done with: >> git format-patch --subject-prefix="PATCH v2" > > Okay, I will be sure to put the patch series in the subject. Thanks for the > feedback. > >> Thanks, >> Luis My pleasure Kevin, Luis ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 0/3] fix issues in prism2usb.c.
On Thu, May 26, 2016 at 01:24:28PM +0100, Luis de Bethencourt wrote: > On 26/05/16 04:22, Kevin McKinney wrote: > > This patchset will address issues reported by checkpatch.pl tool > > for file prism2usb.c. > > > > Kevin McKinney (3): > > Staging: wlan-ng: fix alignment not matching. > > Staging: wlan-ng: fix unnecessary parentheses. > > Staging: wlan-ng: fix comparison to NULL issue. > > > > drivers/staging/wlan-ng/prism2usb.c | 14 +++--- > > 1 file changed, 7 insertions(+), 7 deletions(-) > > > > Hello Kevin, > > Thanks for fixing the issue. > > A note for the future, when you resend a patch series you should > mark it as a new version. So the system maintainer knows the evolution > of the patchset and to drop the earlier versions in his process. > > Look at this random example I just found: > https://lkml.org/lkml/2016/5/26/161 > > If you are using git format-patch, this is done with: > git format-patch --subject-prefix="PATCH v2" Okay, I will be sure to put the patch series in the subject. Thanks for the feedback. > Thanks, > Luis ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 0/3] fix issues in prism2usb.c.
On 26/05/16 04:22, Kevin McKinney wrote: > This patchset will address issues reported by checkpatch.pl tool > for file prism2usb.c. > > Kevin McKinney (3): > Staging: wlan-ng: fix alignment not matching. > Staging: wlan-ng: fix unnecessary parentheses. > Staging: wlan-ng: fix comparison to NULL issue. > > drivers/staging/wlan-ng/prism2usb.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > Hello Kevin, Thanks for fixing the issue. A note for the future, when you resend a patch series you should mark it as a new version. So the system maintainer knows the evolution of the patchset and to drop the earlier versions in his process. Look at this random example I just found: https://lkml.org/lkml/2016/5/26/161 If you are using git format-patch, this is done with: git format-patch --subject-prefix="PATCH v2" Thanks, Luis ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH 0/3] fix issues in prism2usb.c.
This patchset will address issues reported by checkpatch.pl tool for file prism2usb.c. Kevin McKinney (3): Staging: wlan-ng: fix alignment not matching. Staging: wlan-ng: fix unnecessary parentheses. Staging: wlan-ng: fix comparison to NULL issue. drivers/staging/wlan-ng/prism2usb.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) -- 2.1.4 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel