Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.

[Greg KH <gregkh@linuxfoundation.org>]

On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
> Logging messages that show some type of 'out of memory' error
> are generally unnecessary as there is a generic message and
> a stack dump done by the memory subsystem.
> 
> These messages generally increase kernel size without much
> added value.
> 
> Problem found by checkpatch.
> 
> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
> 
> Changes in v4:
>   -Include v2 and v3 in the patch.
> 
> Changes in v3:
>   -Make the commit message more appropriate.
> 
> changes in v2:
>   -Original patch was not sent to the mailing list.
>    Also, add a tab before the return statement.
> 
>  drivers/staging/rtlwifi/rc.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
> index 65de0c7..779a5c6 100644
> --- a/drivers/staging/rtlwifi/rc.c
> +++ b/drivers/staging/rtlwifi/rc.c
> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
> 	struct rtl_rate_priv *rate_priv;
> 
> 	rate_priv = kzalloc(sizeof(*rate_priv), gfp);
> -	if (!rate_priv) {
> -		pr_err("Unable to allocate private rc structure\n");
> +	if (!rate_priv)
> 		return NULL;
> -	}
> 
> 	rtlpriv->rate_priv = rate_priv;
> 

This patch is sent corrupted (leading space is gone), and git doesn't
like it.  Can you please fix it up and resend?

thanks,

greg k-h