From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D0D7EC07E94 for ; Fri, 4 Jun 2021 09:08:56 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 99BFB613EA for ; Fri, 4 Jun 2021 09:08:56 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 99BFB613EA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=iXZdLdHPzUWDpUj+Ip1y5JyQ2dcqz4kHFXTITGiu3yE=; b=q6H2oxCcgr4gI+vXTKjz8VFzfF L28SfrcdHd3HwzRwnA/htb8qV3b9YnRC2tKxG3ia61u8LAeH4sYeHKeZ/nTyQ0KzqXeT43RzyQ7l4 tJVzUfAutseDzSIB4UzQW6jNZl6XsKwYCXD/o8by5WNFlQg1hMT286tCj9kH66ZxMMLUFts1EJ8uv wqvoPe+cwc7Ig+nDMto3sRXflmvEV9dm10bcMJnqyOPBGNf2KccZvNlRB3vJwsYjqMoud5g6N9IGx VDbxk65lf8cfpV+wquntF5mfrrJqDrDZdOIZdLprKTAXw8faJFYGzjrk49lTb6g2/ugKqE8jqR1Yf upi60E+Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lp5mt-00CUDz-Nl; Fri, 04 Jun 2021 09:06:45 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lp5mf-00CU8B-Iz for linux-arm-kernel@lists.infradead.org; Fri, 04 Jun 2021 09:06:34 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 2DAE1613E9; Fri, 4 Jun 2021 09:06:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1622797588; bh=hz2yLbMIN04mVcnLPlTJJ4FivtW2CO0lFJxtHSFzefs=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=PbAm2aA/Wwm/Dl33Eb1EKv08lX1DNWUdWwUHDBUVLfD7YzCy9/SCHnuBTESQPrFJ+ 4fHlWbNmmZkhvZDbygNnt9Xe+rXW1EJHNgvfWG7yASigR/Fcx9S1jCe1coMANKWDyP 6/sBsH0GA14NoSmdoGngZOEbCONbHBGTAyD3exMvQEeqfznADsO2gv5a9vIVJN/OPO Z7LhtpZa3z5cf7Zrq3PT7XFXkcGiUwiaMKYzbX8uLTkM/aKbQJuy0h/wWZQKjB45mt c48nD7nAP9H5ccTQKBKavMUgz3Z5mRWkBMqOnqjEZCoeQ7Upd08w6USP0rbgz0koz3 +IZQpYPUFqOYQ== Date: Fri, 4 Jun 2021 11:06:19 +0200 From: Wolfram Sang To: Lee Jones Cc: linux-kernel@vger.kernel.org, Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Marek Vasut , linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 09/16] i2c: busses: i2c-mxs: Demote barely half complete kernel-doc header Message-ID: Mail-Followup-To: Wolfram Sang , Lee Jones , linux-kernel@vger.kernel.org, Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Marek Vasut , linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20210520190105.3772683-1-lee.jones@linaro.org> <20210520190105.3772683-10-lee.jones@linaro.org> <20210604084710.GG2435141@dell> MIME-Version: 1.0 In-Reply-To: <20210604084710.GG2435141@dell> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210604_020629_698442_1DDF3A7E X-CRM114-Status: GOOD ( 13.05 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============2375501425252745593==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2375501425252745593== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="HVpMPg5ody15+ki9" Content-Disposition: inline --HVpMPg5ody15+ki9 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline > IMHO, we wouldn't want to foster the impression that it's okay to > provide a non-determined effort, safe in the knowledge that someone > will come along later and finish the job for them at a later date. Right. The first lesson from that is that maintainers should require documentation of the fields when they get added. This was my oversight because it was back then not reported by checkers, probably. I am sorry. It annoys me, too. If I notice that someone updates a driver which doc-errors, then I ask if that could be fixed by this person, too. It usually works. Not for drivers without attention, of course. But this is why I don't mind doc-errors to stay. If this is considered problematic, then I'd suggest to remove the kernel doc headers like you did, but add a comment like: * FIXME: add missing fields and reenable kernel-doc To make sure, it is obvious even by glimpsing through the code that there is work needed. Can we agree on that? --HVpMPg5ody15+ki9 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmC57QgACgkQFA3kzBSg KbYAOQ/8C+scGQC4p3+r5I80G/gbqE/SSyxiCum+NVbnGI56Wg7w/LZrg41lB9Cc 50vbSPBv7NTcoewFNfFY+ISV325Y0oeyuAuwF1BVJ/IYvc50cI18PxPYBD5Vn81K ll1PI/Ue+j/cCRg3a+KpdpNH5/+jQwO5P8tI4m9UgybAXMp+EzgQ+t+MZgXRzsLS aiQSvmZh1AGgQyo0P0b4GoocuPTQ+tF90+DSH/gVwE17rmN30qXtTm6N6m2kxoyT 7bbdLfOOKaOZPTznB93ZiKPxgz8MTQi3UIWFj3Hid7gbKKYj5+3vjlbLl4CM6eoF 8acl8OdtA4vSM7OmUcN1PWMDGVUyKNyfAGpLwU+5aR2Js9WC0CPg6STfsjW4il1d /aFLozmv5erxYVSWVs5L6Ss1/PJok5R5HmyLt5DMjBFemjyUYz3H3+En7LEELnr4 WGi7qUaEfsiWJ8av36uyrZlakEo9mfDZwril4XB7skNp7LiK866+w61jdvgVoFbN AQzTWIrkycPS7awZsrimMOmq5hnTY1zDL3nc3LI7O+04GmzBzNxNIo7x6IF3gajI k+BoNmoXf84C3l+/wLeK3gqcSADqTteZkkBzJt0M5wIYSk3sMh8ab3I/IBQUJkkR pnNTLTV5Tl+kdwkEh4PK+FI+SOpQIjqmZVAFGRmUFF7WK4IBW04= =lxjk -----END PGP SIGNATURE----- --HVpMPg5ody15+ki9-- --===============2375501425252745593== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2375501425252745593==--