From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932859AbcFHPjA (ORCPT ); Wed, 8 Jun 2016 11:39:00 -0400 Received: from smtprelay0126.hostedemail.com ([216.40.44.126]:45699 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932293AbcFHPi7 (ORCPT ); Wed, 8 Jun 2016 11:38:59 -0400 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,:::::::::::::::::,RULES_HIT:41:69:334:355:368:369:379:541:960:973:981:988:989:1260:1277:1311:1313:1314:1345:1359:1373:1437:1515:1516:1518:1534:1542:1593:1594:1711:1714:1730:1747:1777:1792:2194:2198:2199:2200:2393:2553:2559:2562:2828:2908:3138:3139:3140:3141:3142:3350:3865:3866:3867:3868:3870:3871:3872:3873:3874:4321:5007:6119:7264:7875:7902:7903:9038:9707:10004:10400:10450:10455:10848:11026:11232:11473:11658:11914:12043:12438:12517:12519:12679:12683:12740:13255:13439:13894:14093:14095:14181:14659:14721:19904:19999:21063:21080,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:1,LUA_SUMMARY:none X-HE-Tag: worm70_3086733ca4c54 X-Filterd-Recvd-Size: 3271 Message-ID: <1465400332.25087.56.camel@perches.com> Subject: kbuild: default n removals? (was: Re: [PATCH v3 4/6] watchdog: add watchdog pretimeout framework) From: Joe Perches To: Vladimir Zapolskiy , Guenter Roeck , Michal Marek Cc: Wim Van Sebroeck , Wolfram Sang , Robin Gong , linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kbuild Date: Wed, 08 Jun 2016 08:38:52 -0700 In-Reply-To: <5758358A.602@mentor.com> References: <1465321127-19522-1-git-send-email-vladimir_zapolskiy@mentor.com> <1465321127-19522-5-git-send-email-vladimir_zapolskiy@mentor.com> <20160607214309.GA17129@roeck-us.net> <57581FB2.10806@mentor.com> <57582363.8050406@roeck-us.net> <5758358A.602@mentor.com> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.18.5.2-0ubuntu3 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org (Adding Michal Marek and linux-kbuild) On Wed, 2016-06-08 at 18:11 +0300, Vladimir Zapolskiy wrote: > On 08.06.2016 16:53, Guenter Roeck wrote: > > On 06/08/2016 06:37 AM, Vladimir Zapolskiy wrote: > > > > > +comment "Watchdog Pretimeout Governors" > > > > > + > > > > > +config WATCHDOG_PRETIMEOUT_GOV > > > > > + bool "Enable watchdog pretimeout governors" > > > > > + default n > > > > I don't think 'default n" is needed. > > > > > > > No strict objections, but probably 'default n' may save quite many > > > lines in defconfigs. > > > > > I always wondered why it would be necessary to say "default n". > > What is the difference between "default n" and no explicit default ? > > > I pointed out that it may have impact on defconfig, but experimentally > it has no effect. > > Users of "make oldconfig" get a prompt in both cases as well. > > Also I haven't found any difference for silentoldconfig, olddefconfig > and alldefconfig, I assume explicit "default n" and "def_bool n" > can be safely dropped. It's not completely clear removals are always appropriate. from: Documentation/kbuild/kconfig-language.txt: ------------------------------------------------------------------ - default value: "default" ["if" ]   A config option can have any number of default values. If multiple   default values are visible, only the first defined one is active.   Default values are not limited to the menu entry where they are   defined. This means the default can be defined somewhere else or be   overridden by an earlier definition.   The default value is only assigned to the config symbol if no other   value was set by the user (via the input prompt above). If an input   prompt is visible the default value is presented to the user and can   be overridden by him.   Optionally, dependencies only for this default value can be added with   "if". ------------------------------------------------------------------ Michal? Do you have an opinion or clarification?