Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753140AbdF2MvF (ORCPT ); Thu, 29 Jun 2017 08:51:05 -0400 Received: from pandora.armlinux.org.uk ([78.32.30.218]:41284 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753109AbdF2Mu6 (ORCPT ); Thu, 29 Jun 2017 08:50:58 -0400 Date: Thu, 29 Jun 2017 13:50:41 +0100 From: Russell King - ARM Linux To: Viresh Kumar Cc: Randy Dunlap , Rafael Wysocki , Vincent Guittot , Rob Herring , Greg Kroah-Hartman , Stephen Boyd , linux-kernel@vger.kernel.org, Mark Brown , rnayak@codeaurora.org, Shiraz Hashim , linux-arm-kernel@lists.infradead.org Subject: Re: [RFC 1/5] drivers: Add boot constraints core Message-ID: <20170629125040.GW4902@n2100.armlinux.org.uk> References: <24921f2a6a86cb9b2b3a1cae86649180bc116a62.1498642745.git.viresh.kumar@linaro.org> <78941b27-c233-837d-9709-b11c01f1ab1f@infradead.org> <20170629035157.GG29665@vireshk-i7> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170629035157.GG29665@vireshk-i7> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 816 Lines: 25 On Thu, Jun 29, 2017 at 09:21:57AM +0530, Viresh Kumar wrote: > On 28-06-17, 08:55, Randy Dunlap wrote: > > On 06/28/2017 03:26 AM, Viresh Kumar wrote: > > > > +config BOOT_CONSTRAINTS > > > + bool "Boot constraints for devices" > > > + default y > > > > Why default y? > > > > As Linus just wrote yesterday: > > > > No. We've tried. The only sensible default (and that I try to enforce) > > is "new featrures default to 'n'" > > Yeah, this should have been n really. Given that the default default is to default to n, you don't need to supply a default that just says what the default default actually is. Please also avoid silly defaults. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.