On Mon, 2012-07-09 at 00:59 +0200, Jesper Juhl wrote: > I missed a few redundant newlines the first time. [] > diff --git a/drivers/staging/vt6656/iwctl.c b/drivers/staging/vt6656/iwctl.c [] > @@ -964,7 +964,6 @@ int iwctl_giwrts(struct net_device *dev, struct iw_request_info *info, > wrq->value = pDevice->wRTSThreshold; > wrq->disabled = (wrq->value >= 2312); > wrq->fixed = 1; > - > return 0; > } I think this is a useful style and these should not be removed. It's akin to a blank line after automatic variable declarations. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel