From: boris brezillon <b.brezillon@overkiz.com> To: Wim Van Sebroeck <wim@iguana.be> Cc: Rob Herring <rob.herring@calxeda.com>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Stephen Warren <swarren@wwwdotorg.org>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Rob Landley <rob@landley.net>, Russell King <linux@arm.linux.org.uk>, Fabio Porcedda <fabio.porcedda@gmail.com>, Nicolas Ferre <nicolas.ferre@atmel.com>, Guenter Roeck <groeck7@gmail.com>, Yang Wenyou <wenyou.yang@atmel.com>, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-watchdog@vger.kernel.org Subject: Re: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt Date: Tue, 29 Oct 2013 14:25:38 +0100 [thread overview] Message-ID: <526FB752.7070708@overkiz.com> (raw) In-Reply-To: <20131029125848.GB5261@spo001.leaseweb.com> On 29/10/2013 13:58, Wim Van Sebroeck wrote: > Hi Boris, > >> I'm sorry for the inconvenience, but I found some bugs in my patch series: >> >> 1) the secs_to_ticks returns an erronous value when 0 is passed as an >> argument >> 2) the calculated heartbeat is too small for some use cases >> (i.e. kexecing a new kernel might trigger a watchdog reset before >> the new kernel >> is able to load the watchdog driver) >> 3) when initializing the watchdog driver, the timer should be configured >> with the min_heartbeat value >> instead of the standard heartbeat value, because we don't for how >> long the timer has been running. >> >> I'll send a new patch fixing those issues. >> I hope it won't bother you :-(. > No problem :-). Just sent me the fixes when they are ready. Already done : https://lkml.org/lkml/2013/10/29/207. Thanks. Best Regards, Boris > > Kind regards, > Wim. >
WARNING: multiple messages have this Message-ID (diff)
From: b.brezillon@overkiz.com (boris brezillon) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt Date: Tue, 29 Oct 2013 14:25:38 +0100 [thread overview] Message-ID: <526FB752.7070708@overkiz.com> (raw) In-Reply-To: <20131029125848.GB5261@spo001.leaseweb.com> On 29/10/2013 13:58, Wim Van Sebroeck wrote: > Hi Boris, > >> I'm sorry for the inconvenience, but I found some bugs in my patch series: >> >> 1) the secs_to_ticks returns an erronous value when 0 is passed as an >> argument >> 2) the calculated heartbeat is too small for some use cases >> (i.e. kexecing a new kernel might trigger a watchdog reset before >> the new kernel >> is able to load the watchdog driver) >> 3) when initializing the watchdog driver, the timer should be configured >> with the min_heartbeat value >> instead of the standard heartbeat value, because we don't for how >> long the timer has been running. >> >> I'll send a new patch fixing those issues. >> I hope it won't bother you :-(. > No problem :-). Just sent me the fixes when they are ready. Already done : https://lkml.org/lkml/2013/10/29/207. Thanks. Best Regards, Boris > > Kind regards, > Wim. >
next prev parent reply other threads:[~2013-10-29 13:51 UTC|newest] Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-10-04 7:24 [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 7:24 ` [PATCH v5 1/4] watchdog: at91sam9_wdt: better watchdog support Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 22:30 ` Guenter Roeck 2013-10-04 22:30 ` Guenter Roeck 2013-10-04 22:30 ` Guenter Roeck 2013-10-04 7:24 ` [PATCH v5 2/4] watchdog: at91sam9_wdt: update device tree doc Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 22:30 ` Guenter Roeck 2013-10-04 22:30 ` Guenter Roeck 2013-10-04 7:24 ` [PATCH v5 3/4] ARM: at91/dt: add sam9 watchdog default options to SoCs Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 22:31 ` Guenter Roeck 2013-10-04 22:31 ` Guenter Roeck 2013-10-04 7:24 ` [PATCH v5 4/4] ARM: at91/dt: add watchdog properties to kizbox board Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 7:24 ` Boris BREZILLON 2013-10-04 22:32 ` Guenter Roeck 2013-10-04 22:32 ` Guenter Roeck 2013-10-04 22:32 ` Guenter Roeck 2013-10-29 7:50 ` [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt Wim Van Sebroeck 2013-10-29 10:31 ` boris brezillon 2013-10-29 10:31 ` boris brezillon 2013-10-29 12:58 ` Wim Van Sebroeck 2013-10-29 13:25 ` boris brezillon [this message] 2013-10-29 13:25 ` boris brezillon 2013-10-29 10:37 ` [PATCH] watchdog: at91sam9_wdt: various fixes Boris BREZILLON 2013-10-29 10:37 ` Boris BREZILLON 2013-10-29 15:45 ` Guenter Roeck 2013-10-29 15:45 ` Guenter Roeck 2013-10-29 16:22 ` boris brezillon 2013-10-29 16:22 ` boris brezillon 2013-10-29 16:43 ` Guenter Roeck 2013-10-29 16:43 ` Guenter Roeck 2013-10-29 17:22 ` boris brezillon 2013-10-29 17:22 ` boris brezillon 2013-10-29 21:27 ` Guenter Roeck 2013-10-29 21:27 ` Guenter Roeck 2013-10-30 6:01 ` b.brezillon 2013-10-30 6:01 ` b.brezillon at overkiz.com 2013-10-31 10:13 ` Jean-Christophe PLAGNIOL-VILLARD 2013-10-31 10:13 ` Jean-Christophe PLAGNIOL-VILLARD 2013-11-03 14:42 ` boris brezillon 2013-11-03 14:42 ` boris brezillon 2013-11-03 16:22 ` Guenter Roeck 2013-11-03 16:22 ` Guenter Roeck 2013-11-03 17:55 ` boris brezillon 2013-11-03 17:55 ` boris brezillon
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=526FB752.7070708@overkiz.com \ --to=b.brezillon@overkiz.com \ --cc=devicetree@vger.kernel.org \ --cc=fabio.porcedda@gmail.com \ --cc=groeck7@gmail.com \ --cc=ijc+devicetree@hellion.org.uk \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-watchdog@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=mark.rutland@arm.com \ --cc=nicolas.ferre@atmel.com \ --cc=pawel.moll@arm.com \ --cc=rob.herring@calxeda.com \ --cc=rob@landley.net \ --cc=swarren@wwwdotorg.org \ --cc=wenyou.yang@atmel.com \ --cc=wim@iguana.be \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.