From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752710AbcHJVms (ORCPT ); Wed, 10 Aug 2016 17:42:48 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:33834 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751722AbcHJVmr (ORCPT ); Wed, 10 Aug 2016 17:42:47 -0400 Date: Wed, 10 Aug 2016 22:42:36 +0100 From: Salah Triki To: Luis de Bethencourt Cc: linux-kernel@vger.kernel.org, akpm@linux-foundation.org, viro@zeniv.linux.org.uk, salah.triki@acm.org Subject: Re: [PATCH 3/4] befs: fix comment style Message-ID: <20160810214236.GB4372@pc> References: <1470780086-11594-1-git-send-email-luisbg@osg.samsung.com> <1470780086-11594-3-git-send-email-luisbg@osg.samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1470780086-11594-3-git-send-email-luisbg@osg.samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 09, 2016 at 11:01:25PM +0100, Luis de Bethencourt wrote: > The description of befs_load_sb was confusing the kernel-doc system since, > because it starts with /**, it thinks it will document the function with > kernel-doc formatting. Which it isn't. > > Fix other comment style issues in the file while we are at it. > > Signed-off-by: Luis de Bethencourt > --- > fs/befs/super.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/fs/befs/super.c b/fs/befs/super.c > index c36745d..80b93c0 100644 > --- a/fs/befs/super.c > +++ b/fs/befs/super.c > @@ -13,13 +13,9 @@ > #include "befs.h" > #include "super.h" > > -/** > +/* > * befs_load_sb -- Read from disk and properly byteswap all the fields > * of the befs superblock > - * > - * > - * > - * > */ > int > befs_load_sb(struct super_block *sb, befs_super_block * disk_sb) > @@ -93,8 +89,8 @@ befs_check_sb(struct super_block *sb) > } > > /* > - * block_shift and block_size encode the same information > - * in different ways as a consistency check. > + * block_shift and block_size encode the same information > + * in different ways as a consistency check. > */ > > if ((1 << befs_sb->block_shift) != befs_sb->block_size) { > -- > 2.5.1 > Signed-off-by: Salah Triki Thanx Salah