Re: [PATCH] staging: rtl8192e: Change fixed function names with "%s:", __func__
On Mon, Aug 21, 2017 at 09:32:35PM +0800, Shurong Zhang wrote: > Replace hard-coded function names in strings with "%s", __func__ in the > dot11d.c file. Issue found by checkpatch.pl. Please always wrap your changelog text at 72 columns. And why did you send this twice? thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8192e: Change fixed function names with "%s: ", __func__
On Fri, Aug 18, 2017 at 11:53:41PM +0800, 张树荣 wrote: > From: Shurong Zhang > > Replace hard-coded function names in strings with "%s", __func__ > in the dot11d.c file. Issue found by checkpatch.pl. > > Signed-off-by: Shurong Zhang > --- > drivers/staging/rtl8192e/dot11d.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) This patch doesn't apply at all, are you sure you generated it correctly? thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8192e: Change fixed function names with "%s: ", __func__
On Thu, Aug 03, 2017 at 09:46:26AM +0800, vergilzhang wrote: > Replace hard-coded function names in strings with "%s", __func__ > in the dot11d.c file. Issue found by checkpatch.pl. > > Signed-off-by: vergilzhang Please use your "legal" name here, and in the From line. Please fix up and resend all of your submitted patches with that change. thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8192e: Change fixed function names with "%s: ", __func__
On Thu, Aug 03, 2017 at 01:59:56AM +0800, vergilzhang wrote: > staging: rtl8192e: Fix coding style issue caught by checkpatch.pl related to > function name in debug print > > Signed-off-by: vergilzhang > --- > drivers/staging/rtl8192e/dot11d.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) You didn't pay attention to the other issues my patch bot sent you about this patch, why not? I'll just wait for you to fix them all and drop this one for now, sorry. greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel