From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751699AbcEJNZb (ORCPT ); Tue, 10 May 2016 09:25:31 -0400 Received: from smtpoutz28.laposte.net ([194.117.213.103]:36279 "EHLO smtp.laposte.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751568AbcEJNZa (ORCPT ); Tue, 10 May 2016 09:25:30 -0400 X-Greylist: delayed 1390 seconds by postgrey-1.27 at vger.kernel.org; Tue, 10 May 2016 09:25:30 EDT Message-ID: <5731DBD9.80804@laposte.net> Date: Tue, 10 May 2016 15:02:17 +0200 From: Sebastian Frias User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Andy Whitcroft , Joe Perches CC: LKML , mason Subject: [PATCH] checkpatch: make the config description warning more explicit Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-VR-SrcIP: 83.142.147.193 X-VR-FullState: 0 X-VR-Score: 0 X-VR-Cause-1: gggruggvucftvghtrhhoucdtuddrfeekledrudeggdeitdcutefuodetggdotefrodftvfcurfhrohhf X-VR-Cause-2: ihhlvgemucfntefrqffuvffgnecuuegrihhlohhuthemucehtddtnecunecujfgurhepkfffhfgfggfv X-VR-Cause-3: ufgtgfesthejrgdttdefjeenucfhrhhomhepufgvsggrshhtihgrnhcuhfhrihgrshcuoehsfhekgees X-VR-Cause-4: lhgrphhoshhtvgdrnhgvtheqnecukfhppeekfedrudegvddrudegjedrudelfeenucfrrghrrghmpehm X-VR-Cause-5: ohguvgepshhmthhpohhuthdphhgvlhhopegludejvddrvdejrddtrddvudegngdpihhnvghtpeekfedr X-VR-Cause-6: udegvddrudegjedrudelfedpmhgrihhlfhhrohhmpehsfhekgeeslhgrphhoshhtvgdrnhgvthdprhgt X-VR-Cause-7: phhtthhopegrphifsegtrghnohhnihgtrghlrdgtohhm X-VR-AvState: No X-VR-State: 0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org checkpatch.pl currently warns if Kconfig options are properly described by checking the length of the config description. This patch is an attempt to make the message less cryptic. Signed-off-by: Sebastian Frias --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 8fc9edd..9afc06f 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2663,7 +2663,7 @@ sub process { } if ($is_start && $is_end && $length < $min_conf_desc_length) { WARN("CONFIG_DESCRIPTION", - "please write a paragraph that describes the config symbol fully\n" . $herecurr); + "description is too short ($length lines), please write at least $min_conf_desc_length lines to describe the config symbol fully\n" . $herecurr); } #print "is_start<$is_start> is_end<$is_end> length<$length>\n"; } -- 2.1.4