Re: [PATCH 1/3] wlan-ng/prism2mgmt.c: added parenthesis to macro

From: Greg Kroah-Hartman
Date: Fri Jan 18 2013 - 16:02:01 EST


On Fri, Jan 18, 2013 at 12:07:30AM +0100, Sebastian Ehrenfels wrote:
> From: Sebastian Wankerl <sisewank@xxxxxxxxxxxxx>
>
> Signed-off-by: Sebastian Wankerl <sisewank@xxxxxxxxxxxxx>
> Signed-off-by: Sebastian Ehrenfels <qi50dube@xxxxxxxxxxxxx>

Why did you do this? What does this fix? Please always say that in the
change log entry part of the patch, empty ones aren't ok.

And why do you have trailing spaces on your signed-off-by: lines?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/