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