From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.gmx.net ([212.227.15.19]:59380 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751111AbaEGUTv (ORCPT ); Wed, 7 May 2014 16:19:51 -0400 Date: Wed, 7 May 2014 22:19:41 +0200 From: Michael Brunner To: gundberg Cc: wim@iguana.be, linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] watchdog: kempld-wdt: Use the correct value when configuring the prescaler with the watchdog Message-ID: <20140507221941.2d6468ae@mail.gmx.de> In-Reply-To: <5359165F.5040102@icomera.com> References: <5359165F.5040102@icomera.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org On Thu, 24 Apr 2014 08:15:40 -0700, Guenter Roeck wrote: > On Thu, Apr 24, 2014 at 03:49:19PM +0200, gundberg wrote: > > Use the prescaler index, rather than its value, to configure the > > watchdog. This will prevent a mismatch with the prescaler used to > > calculate the cycles. > > > > Signed-off-by: Per Gundberg > > Good catch. Looks correct to me. Michael, any comments ? > > Reviewed-by: Guenter Roeck Setting the prescaler incorrectly this way can cause the system to reboot nearly immediately when activating the watchdog - not good. This should also be applied to the stable revisions as this code is included since 3.11. Reviewed-by: Michael Brunner Tested-by: Michael Brunner Thanks, Michael