Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp5129331pxj; Wed, 26 May 2021 03:36:58 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzBEzoHUS/UfFuOpxYJ+zA6HXII9WqMgRRHqBkgur5VkOacmbEfZTzF7vKpoE9F6cGDIU7O X-Received: by 2002:a92:c56a:: with SMTP id b10mr23385731ilj.17.1622025418496; Wed, 26 May 2021 03:36:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1622025418; cv=none; d=google.com; s=arc-20160816; b=dzNE57o1p6aoVHT49lNAyTnCFZOSAM3ol4hgYMdp6YWw/FvAOa5EzzHBOaLzBiytPS Xt4F0crch7TlXTRHXngeh9U6a8t6XDzi/QZPxh0cUrPLVkb0ggtp6/3/wknnVzjWmKzo Uc059bHmhf22xStL9dkP+Jy23rhU8XADoQZrc1ho9tCxxd+Y2omcbXybJn/a4MRNjHIg ULkoTNJvVGZh92bDQtnvuWlX+PLVYlMqGSjjsJKRiURimAlsbVYEEMlBkF4/SKZ7dOOk N8Nr9DTlFgy9ZAMK+JLRkJAq4VRkc/nIHlIHHD07oDRTfmnWyilOS4UNGNLl29fUUS8G rwWw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent:references:in-reply-to :date:cc:to:from:subject:message-id:dkim-signature; bh=RTL0YUJC5bTgHB3rr9d5JzhmkX+MPRCkX3Z5f/ZfoIw=; b=V6ZGORr6wheFIa4Hv+2VtlkU+DueWCuOYX1O7KbF4xSCknhqOZp0/px98KfHNw6pNl M9Q8FYlTCplyTLwU55hvDRcHn1hGKHzwmUCrMkdwYYjlE8LOSSWc35z2/HDeK+EraqUb 6CESpDTM+RjQwFSPibHRKmZOVJW4wHq27d/lZzQXkOZKgVHuFMVpfI5NpznZ6HbXd0g0 qSgbmj2moMj5pUFJ8q35ts+ep6pwLbPikvXKY/LddLGKwpdCmuHi1RzyMaZ4QcyEMTi4 TLKqIIFQfM1FPmFkzJJwNqyY9hgqZ24WnPU6DZB0ZSrDL9Bc5e57NiYzVsCuPpI2cO+m Q2qA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@igalia.com header.s=20170329 header.b="H/ZH+V7f"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i14si19136297ilq.76.2021.05.26.03.36.44; Wed, 26 May 2021 03:36:58 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@igalia.com header.s=20170329 header.b="H/ZH+V7f"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233993AbhEZKhc (ORCPT + 99 others); Wed, 26 May 2021 06:37:32 -0400 Received: from fanzine.igalia.com ([178.60.130.6]:34377 "EHLO fanzine.igalia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233969AbhEZKhc (ORCPT ); Wed, 26 May 2021 06:37:32 -0400 X-Greylist: delayed 1758 seconds by postgrey-1.27 at vger.kernel.org; Wed, 26 May 2021 06:37:31 EDT DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=MIME-Version:Content-Type:References:In-Reply-To:Date:Cc:To:From:Subject:Message-ID; bh=RTL0YUJC5bTgHB3rr9d5JzhmkX+MPRCkX3Z5f/ZfoIw=; b=H/ZH+V7fPt0bFbpnORL3NeeKS1NIF7F8GL9L5BF0Up1yA1EKow1Cgxk+rPe765clKv6DWQ56QB4fFYkhrH0kIk0voXlqMHd0LOVx5UiLeh0ZbZuZHwnZn8f3nkqvp8SxNsOBDGWLN3pRZvl1pM2oMggNf4AcWDQuNWZ1hCVFNBO0FHpt5tpSI1a3rKxGGBQNcQnl8BkuYETvqYMLgHN4pZ3kkDvsFhnR68vFPZPChvMSpB+A49rjIr6aVT3EpQQ0Eu9tiu7up+J3Jd+KRRu5eouw5sZLzwoyVIHyO/5BCfk/BdVDF3sfBRS4jd9BzFCE2WK24qUd/TXv9IvzFxLByQ==; Received: from 1.pool85-50-22.dynamic.orange.es ([85.50.22.1] helo=[192.168.1.120]) by fanzine.igalia.com with esmtpsa (Cipher TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim) id 1llqQq-00035i-SG; Wed, 26 May 2021 12:06:34 +0200 Message-ID: Subject: Re: [PATCH] ipac: ipoctal: fix kernel-doc syntax and remove filename from file headers From: Samuel Iglesias =?ISO-8859-1?Q?Gons=E1lvez?= To: Aditya Srivastava Cc: lukas.bulwahn@gmail.com, rdunlap@infradead.org, linux-kernel-mentees@lists.linuxfoundation.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, jens.taprogge@taprogge.org, gregkh@linuxfoundation.org, industrypack-devel@lists.sourceforge.net Date: Wed, 26 May 2021 12:05:30 +0200 In-Reply-To: <20210522121944.11182-1-yashsri421@gmail.com> References: <20210522121944.11182-1-yashsri421@gmail.com> Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="=-f9pVxU9gW8q5Fs96aXmZ" User-Agent: Evolution 3.38.3-1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-f9pVxU9gW8q5Fs96aXmZ Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Acked-by: Samuel Iglesias Gonsalvez Thanks, Sam On Sat, 2021-05-22 at 17:49 +0530, Aditya Srivastava wrote: > The opening comment mark '/**' is used for highlighting the beginning > of > kernel-doc comments. > The header for drivers/ipack/devices/ipoctal follows this syntax, but > the > content inside does not comply with kernel-doc. >=20 > This line was probably not meant for kernel-doc parsing, but is > parsed > due to the presence of kernel-doc like comment syntax(i.e, '/**'), > which > causes unexpected warning from kernel-doc. >=20 > For e.g., running scripts/kernel-doc -none on > drivers/ipack/devices/ipoctal.h emits: > warning: This comment starts with '/**', but isn't a kernel-doc > comment. Refer Documentation/doc-guide/kernel-doc.rst > =C2=A0* ipoctal.h >=20 > Provide a simple fix by replacing this occurrence with general > comment > format, i.e. '/*', to prevent kernel-doc from parsing it. >=20 > Also remove the redundant file name from the comment headers. >=20 > Signed-off-by: Aditya Srivastava > --- > =C2=A0drivers/ipack/devices/ipoctal.c | 4 +--- > =C2=A0drivers/ipack/devices/ipoctal.h | 6 ++---- > =C2=A02 files changed, 3 insertions(+), 7 deletions(-) >=20 > diff --git a/drivers/ipack/devices/ipoctal.c > b/drivers/ipack/devices/ipoctal.c > index 3940714e4397..2a3a94f72dfb 100644 > --- a/drivers/ipack/devices/ipoctal.c > +++ b/drivers/ipack/devices/ipoctal.c > @@ -1,7 +1,5 @@ > =C2=A0// SPDX-License-Identifier: GPL-2.0-only > -/** > - * ipoctal.c > - * > +/* > =C2=A0 * driver for the GE IP-OCTAL boards > =C2=A0 * > =C2=A0 * Copyright (C) 2009-2012 CERN (www.cern.ch) > diff --git a/drivers/ipack/devices/ipoctal.h > b/drivers/ipack/devices/ipoctal.h > index 75f83ba774a4..773dc41bd667 100644 > --- a/drivers/ipack/devices/ipoctal.h > +++ b/drivers/ipack/devices/ipoctal.h > @@ -1,9 +1,7 @@ > =C2=A0/* SPDX-License-Identifier: GPL-2.0-only */ > -/** > - * ipoctal.h > - * > +/* > =C2=A0 * driver for the IPOCTAL boards > - > + * > =C2=A0 * Copyright (C) 2009-2012 CERN (www.cern.ch) > =C2=A0 * Author: Nicolas Serafini, EIC2 SA > =C2=A0 * Author: Samuel Iglesias Gonsalvez --=-f9pVxU9gW8q5Fs96aXmZ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEQP+ZAvaXWkfuKXiEf/S6MvF9w0MFAmCuHWoACgkQf/S6MvF9 w0OLgg/+LV3KqR1htIZVhhQPi2+KnjqBI6fjsF6pm0aYa6P0zRf8kqSCM/Z9Gk2V ntTgqBWOgKdlN5hyRRZnkzJsPZnqRAeglyZbJ2wfbfYE8G9hrBL75ZUmUJf/Bw2T 9HxvMHNOZM2aTG1avIHmW6J++GOiCN1ptucjHS4OEGA3cadyO8zYCO66fWH1nLtO ls0Wzdg60OjiMtBBISFXOCgUkJNLIs1L9ug8IZIXbenrKbOhnTwnjzyy80zX44Lc MuLAjABIixSFw6KDR+uGlJVqcNge6nHJmEze3A3b5QDrYE/1mQnuzwLjKfJTj5OU lEFLvtDr9YmnpkRICwwn2X6n0mgSIAdNnVUP2JaVE201AsUoXWdqN2AsDLqCjE+W 6lbIt51EfRQdSI7YKm+YW2smSTe/4nKvbUDFsQmLWGjgijt6X9QSxiCdKJxlKp5L HsHVtH60mflOlRTyHWI3kiNUIMhP51HOt3B+DhmVaszuWhE24FAS9cj+nPPCWPDk JorrS6sxpzIRiuMV4maIfnZDiFpBS2UHzJeDmJAleBq+eU1qo9CU9huwtbzp2aY8 p0+XRTPrr8bG2h2XfOWxpghwgbz8SuVB03PRdWKZN5q99u/XUyeRgqayDH9KWBrv ygPsw2ZvbCj9Jaa4MAjAaNz8CqfPkTz+ri+/tKpleotJMhznMGo= =OK0d -----END PGP SIGNATURE----- --=-f9pVxU9gW8q5Fs96aXmZ--