Re: [PATCH] staging: rtl8192e: Change fixed function names with "%s:", __func__

2017-08-22 Thread Greg Kroah-Hartman
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__

2017-08-20 Thread Greg Kroah-Hartman
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__

2017-08-17 Thread Greg Kroah-Hartman
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__

2017-08-02 Thread Greg Kroah-Hartman
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