On Thu, Jun 19, 2014 at 09:20:15PM +0200, Johannes Stadlinger wrote:
> This patch inserts a blank line after a declaration to avoid checkpatch
> warning.
>
> Signed-off-by: Johannes Stadlinger <johannes.stadlin...@fau.de>
> Signed-off-by: Maximilian Eschenbacher <maximilian@eschenbacher.email>
> CC: linux-ker...@i4.cs.fau.de
> CC: Greg Kroah-Hartman <gre...@linuxfoundation.org>
> CC: Josh Triplett <j...@joshtriplett.org>
> CC: Himangi Saraogi <himangi...@gmail.com>
> CC: Vitaly Osipov <vitaly.osi...@gmail.com>
> CC: de...@driverdev.osuosl.org
> CC: linux-ker...@vger.kernel.org
This is one case where I think checkpatch is just wrong; this doesn't
make the code any more (or less) readable.
Meh-by: Josh Triplett <j...@joshtriplett.org>
> drivers/staging/wlan-ng/prism2mgmt.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/staging/wlan-ng/prism2mgmt.c
> b/drivers/staging/wlan-ng/prism2mgmt.c
> index f90f7da..e6a82d3 100644
> --- a/drivers/staging/wlan-ng/prism2mgmt.c
> +++ b/drivers/staging/wlan-ng/prism2mgmt.c
> @@ -190,6 +190,7 @@ int prism2mgmt_scan(wlandevice_t *wlandev, void *msgp)
> word = 0;
> for (i = 0; i < msg->channellist.data.len; i++) {
> u8 channel = msg->channellist.data.data[i];
> +
> if (channel > 14)
> continue;
> /* channel 1 is BIT 0 ... channel 14 is BIT 13 */
> --
> 1.9.1
>
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
