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;
From:   Daniel Axtens <dja@axtens.net>
To:     Randy Dunlap <rdunlap@infradead.org>, linux-kernel@vger.kernel.org
Cc:     Randy Dunlap <rdunlap@infradead.org>, linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH] selftests: powerpc: unmark non-kernel-doc comments
In-Reply-To: <20210325200820.16594-1-rdunlap@infradead.org>
References: <20210325200820.16594-1-rdunlap@infradead.org>
Date:   Fri, 26 Mar 2021 15:50:25 +1100
Message-ID: <87a6qq8qni.fsf@linkitivity.dja.id.au>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Randy Dunlap <rdunlap@infradead.org> writes:

> Drop the 'beginning of kernel-doc' notation markers (/**)
> in places that are not in kernel-doc format.

This looks good to me. Arguably we don't need the comments at all, but
it doesn't seem to hurt to keep them.

checkpatch is OK with the entire file, so there's nothing else we'd
really want to clean up while you're doing cleanups.

Reviewed-by: Daniel Axtens <dja@axtens.net>

Kind regards,
Daniel

>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Cc: linuxppc-dev@lists.ozlabs.org
> ---
>  tools/testing/selftests/powerpc/tm/tm-trap.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> --- linux-next-20210323.orig/tools/testing/selftests/powerpc/tm/tm-trap.c
> +++ linux-next-20210323/tools/testing/selftests/powerpc/tm/tm-trap.c
> @@ -66,7 +66,7 @@ void trap_signal_handler(int signo, sigi
>  	/* Get thread endianness: extract bit LE from MSR */
>  	thread_endianness = MSR_LE & ucp->uc_mcontext.gp_regs[PT_MSR];
>  
> -	/***
> +	/*
>  	 * Little-Endian Machine
>  	 */
>  
> @@ -126,7 +126,7 @@ void trap_signal_handler(int signo, sigi
>  		}
>  	}
>  
> -	/***
> +	/*
>  	 * Big-Endian Machine
>  	 */
>