From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752606AbbJLN4d (ORCPT ); Mon, 12 Oct 2015 09:56:33 -0400 Received: from mx-guillaumet.finsecur.com ([91.217.234.131]:37715 "EHLO guillaumet.finsecur.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752206AbbJLN40 (ORCPT ); Mon, 12 Oct 2015 09:56:26 -0400 Date: Mon, 12 Oct 2015 15:56:22 +0200 From: Sylvain Rochet To: "Yang, Wenyou" Cc: Alexandre Belloni , Guenter Roeck , Boris BREZILLON , "linux-kernel@vger.kernel.org" , "Ferre, Nicolas" , "Desroches, Ludovic" , "linux-arm-kernel@lists.infradead.org" , Wim Van Sebroeck Message-ID: <20151012135622.GA6845@gradator.net> References: <1444340074-15437-1-git-send-email-sylvain.rochet@finsecur.com> <1444340074-15437-7-git-send-email-sylvain.rochet@finsecur.com> <20151012075001.GB3558@piout.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: 172.16.8.13 X-SA-Exim-Mail-From: sylvain.rochet@finsecur.com Subject: Re: [PATCH 6/6] watchdog: sama5d4: try to set timeout from device tree first X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on guillaumet.finsecur.com) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Wenyou, On Mon, Oct 12, 2015 at 08:12:42AM +0000, Yang, Wenyou wrote: > > -----Original Message----- > > From: Alexandre Belloni [mailto:alexandre.belloni@free-electrons.com] > > Sent: 2015年10月12日 15:50 > > To: Sylvain Rochet > > Cc: Guenter Roeck; Boris BREZILLON; linux-kernel@vger.kernel.org; Ferre, > > Nicolas; Desroches, Ludovic; linux-arm-kernel@lists.infradead.org; Yang, > > Wenyou; Wim Van Sebroeck > > Subject: Re: [PATCH 6/6] watchdog: sama5d4: try to set timeout from device tree > > first > > > > Hi Sylvain, > > > > The rest of the series looks good to me, one comment below: > > > > On 08/10/2015 at 23:34:34 +0200, Sylvain Rochet wrote : > > > watchdog_init_timeout function doesn't try to get the "timeout-sec" DT > > > property if timeout_parm is not zero. This change makes this DT > > > property working for the sama5d4 watchdog driver. > > > > > > > While I'm not sure of the feasibility, I think that the module parameter should > > override the DT property. > > The patch should be right, the DT property overrides the module > parameter. > > If the DT property is not a valid value, it uses the default value, > initialized with the module parameter at the beginning of probe. Well, the principle of least surprise applied here means if you load the module with a timeout argument, you expect the timeout argument to be used and not the dt one. As such, it makes more sense to have the parameter value takes precedence over the dt value. Sylvain