Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933559AbdGKPum (ORCPT ); Tue, 11 Jul 2017 11:50:42 -0400 Received: from ns1.pc-advies.be ([85.17.2.162]:49016 "EHLO spo001.leaseweb.nl" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S933465AbdGKPuk (ORCPT ); Tue, 11 Jul 2017 11:50:40 -0400 Date: Tue, 11 Jul 2017 17:50:33 +0200 From: Wim Van Sebroeck To: Guenter Roeck Cc: Rasmus Villemoes , Jonathan Corbet , Sebastian Reichel , esben.haabendal@gmail.com, Alan Cox , linux-watchdog@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [v6,3/3] watchdog: introduce CONFIG_WATCHDOG_OPEN_TIMEOUT Message-ID: <20170711155033.GB31924@spo001.leaseweb.nl> References: <1496134608-7375-4-git-send-email-rasmus.villemoes@prevas.dk> <20170708151554.GA3816@roeck-us.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170708151554.GA3816@roeck-us.net> User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1134 Lines: 27 Hi Guenter, > > This allows setting a default value for the watchdog.open_timeout > > commandline parameter via Kconfig. > > > > Some BSPs allow remote updating of the kernel image and root file > > system, but updating the bootloader requires physical access. Hence, if > > one has a firmware update that requires relaxing the > > watchdog.open_timeout a little, the value used must be baked into the > > kernel image itself and cannot come from the u-boot environment via the > > kernel command line. > > > > Being able to set the initial value in .config doesn't change the fact > > that the value on the command line, if present, takes precedence, and is > > of course immensely useful for development purposes while one has > > console acccess, as well as usable in the cases where one can make a > > permanent update of the kernel command line. > > > > Signed-off-by: Rasmus Villemoes > > Wim, any thoughts on making this configurable ? I used to be opposed to it, > but it does seem to make some sense to me now after thinking about it. I will look at it later this week. Kind regards, Wim.