From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752598AbdJaILx (ORCPT ); Tue, 31 Oct 2017 04:11:53 -0400 Received: from mga03.intel.com ([134.134.136.65]:12871 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751969AbdJaILt (ORCPT ); Tue, 31 Oct 2017 04:11:49 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,323,1505804400"; d="scan'208";a="915741908" From: Jani Nikula To: Matthew Wilcox Cc: Masahiro Yamada , Jonathan Corbet , "open list\:DOCUMENTATION" , Linux Kernel Mailing List , Michal Marek , Linux Kbuild mailing list , Matthew Wilcox Subject: Re: [PATCH] Check all .c files for bad kernel-doc comments In-Reply-To: <20171030205134.GA29718@bombadil.infradead.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20171027194149.14328-1-willy@infradead.org> <20171030134007.GA2846@bombadil.infradead.org> <87mv48fytl.fsf@intel.com> <20171030205134.GA29718@bombadil.infradead.org> Date: Tue, 31 Oct 2017 10:13:10 +0200 Message-ID: <8760avg2g9.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 30 Oct 2017, Matthew Wilcox wrote: > On Mon, Oct 30, 2017 at 05:19:18PM +0200, Jani Nikula wrote: >> Related, there was also a script to do reStructuredText lint style >> checks in addition to the kernel-doc checks using make CHECK and >> C=1. See http://mid.mail-archive.com/87h98quc1w.fsf@intel.com > > I don't really care which patch goes in. If I understand your python > script correctly, it relies on having various python packages installed. > Unless we're going to switch kernel-doc over to being written in python, > I'd prefer to not require additional dependencies. I think your patch has a much better chance of getting enabled by default in the long run, so I'd prefer that. I've also kind of dropped the ball on my script... but thought it might be interesting. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center