Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 11 Mar 2021 17:38:40 +0100
From:   Fabio Aiuto <fabioaiuto83@gmail.com>
To:     Joe Perches <joe@perches.com>
Cc:     gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: rtl8723bs: align comments
Message-ID: <20210311163838.GA5244@agape.jhs>
References: <20210310153717.GA5741@agape.jhs>
 <20210310174830.GM2087@kadam>
 <379d4b58d82be1f871924c89e0db7bab367564fb.camel@perches.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <379d4b58d82be1f871924c89e0db7bab367564fb.camel@perches.com>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk

On Thu, Mar 11, 2021 at 01:15:34AM -0800, Joe Perches wrote:
> On Wed, 2021-03-10 at 20:48 +0300, Dan Carpenter wrote:
> > You need to have a space character after the '*'.
> 
> Perhaps YA checkpatch test...
> ---
>  scripts/checkpatch.pl | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index f42e5ba16d9b..0de553d52605 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -3876,6 +3876,21 @@ sub process {
>  			}
>  		}
>  
> +# Independent comment lines should have a space after the comment initiator
> +		if ($line =~ /^\+[ \t]*($;+)/) {		#leading comment
> +			my $comment = trim(substr($rawline, $-[1], $+[1] - $-[1]));
> +			if ($comment =~ m{^(/\*|\*/|\*|//)(.*)}) {
> +				my $init = $1;
> +				my $rest = $2;
> +				if ($init =~ m{^(?:/\*|\*|//)$} &&
> +				    $rest ne '' &&
> +				    $rest !~ /^[\s\*=\-]/) {
> +					WARN("COMMENT_STYLE",
> +					     "Comments generally use whitespace after the comment initiator\n" . $herecurr);
> +				}
> +			}
> +		}
> +
>  # check for missing blank lines after struct/union declarations
>  # with exceptions for various attributes and macros
>  		if ($prevline =~ /^[\+ ]};?\s*$/ &&
> 

Thank you Joe, it works fine. I've just tested it for my particular need but 
I hope it's going to be merged in the mainline if other people will find it useful:)