Re: [PATCH 06/15] staging: unisys: fix spacing in visorchipset.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 30, 2015 at 02:46:06PM -0500, Benjamin Romer wrote:
> From: Erik Arfvidson <erik.arfvidson@xxxxxxxxxx>
> 
> This patch fixes the following checkpatch warning:
> spaces preferred around that ‘*’ or ‘|’
> 
> Signed-off-by: Erik Arfvidson <erik.arfvidson@xxxxxxxxxx>
> Signed-off-by: Benjamin Romer <benjamin.romer@xxxxxxxxxx>
> ---
>  drivers/staging/unisys/visorbus/visorchipset.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
<snip>
> @@ -522,8 +521,8 @@ parser_string_get(struct parser_context *ctx)
>  		}
>  	if (value_length < 0)	/* '\0' was not included in the length */
>  		value_length = nscan;
> -	value = kmalloc(value_length + 1, GFP_KERNEL|__GFP_NORETRY);
> -	if (value == NULL)
> +	value = kmalloc(value_length + 1, GFP_KERNEL | __GFP_NORETRY);
> +	if (!value)

This should have been there in your 7/15 patch.

regards
sudip
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux