Re: [PATCH] Staging:drivers:wlan_ng: code style issues

From: Bogicevic Sasa
Date: Mon Oct 26 2015 - 18:22:15 EST


On 10/26, Bogicevic Sasa wrote:
Thanks for the review just sent the new patch,
since change is very trivial I believe one liner like "fix all coding
style issues in that and that file" is descriptive enough. Don't know the
best practice so please be patient, I am new at this
On 10/27, Greg KH wrote:
On Mon, Oct 26, 2015 at 05:36:55PM +0100, Bogicevic Sasa wrote:
Fixed coding style issues
Signed-off-by: Bogicevic Sasa <brutallesale@xxxxxxxxx>

What code style issues? Please be very specific. Also be sure you are
only doing one "thing" per patch, and no, "fix all coding style issues"
is not one thing...

And we need a blank line before your signed-off-by: line.

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/