Re: [PATCH] staging: rtl8192u: Fix block comments use * on subsequent lines in r8192U_wx.c

From: Julia Lawall
Date: Mon Feb 01 2016 - 02:01:14 EST




On Mon, 1 Feb 2016, Byeoungwook Kim wrote:

> clean up checkpatch warning:
> WARNING: Block comments use * on subsequent lines
>
> Signed-off-by: Byeoungwook Kim <quddnr145@xxxxxxxxx>
> ---
> drivers/staging/rtl8192u/r8192U_wx.c | 38 +++++++++++++++++++-----------------
> 1 file changed, 20 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c
> index 4911fef..f828e64 100644
> --- a/drivers/staging/rtl8192u/r8192U_wx.c
> +++ b/drivers/staging/rtl8192u/r8192U_wx.c
> @@ -1,21 +1,23 @@
> -/*
> - This file contains wireless extension handlers.
> -
> - This is part of rtl8180 OpenSource driver.
> - Copyright (C) Andrea Merello 2004-2005 <andrea.merello@xxxxxxxxx>
> - Released under the terms of GPL (General Public Licence)
> -
> - Parts of this driver are based on the GPL part
> - of the official realtek driver.
> -
> - Parts of this driver are based on the rtl8180 driver skeleton
> - from Patric Schenke & Andres Salomon.
> -
> - Parts of this driver are based on the Intel Pro Wireless 2100 GPL driver.
> -
> - We want to thank the Authors of those projects and the Ndiswrapper
> - project Authors.
> -*/
> +/******************************************************************************

Is the row of *s needed?

julia

> + *
> + * This file contains wireless extension handlers.
> + *
> + * This is part of rtl8180 OpenSource driver.
> + * Copyright (C) Andrea Merello 2004-2005 <andrea.merello@xxxxxxxxx>
> + * Released under the terms of GPL (General Public Licence)
> + *
> + * Parts of this driver are based on the GPL part
> + * of the official realtek driver.
> + *
> + * Parts of this driver are based on the rtl8180 driver skeleton
> + * from Patric Schenke & Andres Salomon.
> + *
> + * Parts of this driver are based on the Intel Pro Wireless 2100 GPL driver.
> + *
> + * We want to thank the Authors of those projects and the Ndiswrapper
> + * project Authors.
> + *
> + *****************************************************************************/
>
> #include <linux/string.h>
> #include "r8192U.h"
> --
> 2.5.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>