From: Wolfram Sang <wsa@the-dreams.de>
To: Anatolij Gustschin <agust@denx.de>
Cc: linux-arm-kernel@lists.infradead.org,
Shawn Guo <shawn.guo@linaro.org>,
linux-watchdog@vger.kernel.org, Wim Van Sebroeck <wim@iguana.be>
Subject: Re: [PATCH] watchdog: imx2_wdt: handle enabled watchdog properly
Date: Fri, 4 Apr 2014 17:20:12 +0200 [thread overview]
Message-ID: <20140404152011.GC5529@katana> (raw)
In-Reply-To: <1396623242-15234-1-git-send-email-agust@denx.de>
[-- Attachment #1: Type: text/plain, Size: 633 bytes --]
On Fri, Apr 04, 2014 at 04:54:01PM +0200, Anatolij Gustschin wrote:
> If the watchdog is already enabled (e.g. started by the boot loader),
> we need to handle this case properly and start reloading the watchdog
> time-out counter as frequently as needed.
>
> Signed-off-by: Anatolij Gustschin <agust@denx.de>
> Cc: Wolfram Sang <wsa@the-dreams.de>
> Cc: Wim Van Sebroeck <wim@iguana.be>
Uh, this driver has not been converted to use the watchdog framework
yet? I'd suggest this should be the first step.
@Wim: Does the watchdog core already handle the case that the wdt might
have been started by the bootloader?
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: wsa@the-dreams.de (Wolfram Sang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] watchdog: imx2_wdt: handle enabled watchdog properly
Date: Fri, 4 Apr 2014 17:20:12 +0200 [thread overview]
Message-ID: <20140404152011.GC5529@katana> (raw)
In-Reply-To: <1396623242-15234-1-git-send-email-agust@denx.de>
On Fri, Apr 04, 2014 at 04:54:01PM +0200, Anatolij Gustschin wrote:
> If the watchdog is already enabled (e.g. started by the boot loader),
> we need to handle this case properly and start reloading the watchdog
> time-out counter as frequently as needed.
>
> Signed-off-by: Anatolij Gustschin <agust@denx.de>
> Cc: Wolfram Sang <wsa@the-dreams.de>
> Cc: Wim Van Sebroeck <wim@iguana.be>
Uh, this driver has not been converted to use the watchdog framework
yet? I'd suggest this should be the first step.
@Wim: Does the watchdog core already handle the case that the wdt might
have been started by the bootloader?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140404/e7bcaf92/attachment.sig>
next prev parent reply other threads:[~2014-04-04 15:20 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-04 14:54 [PATCH] watchdog: imx2_wdt: handle enabled watchdog properly Anatolij Gustschin
2014-04-04 14:54 ` Anatolij Gustschin
2014-04-04 15:20 ` Wolfram Sang [this message]
2014-04-04 15:20 ` Wolfram Sang
2014-04-04 15:41 ` Guenter Roeck
2014-04-04 15:41 ` Guenter Roeck
2014-04-04 20:07 ` Wim Van Sebroeck
2014-05-26 19:15 ` Wim Van Sebroeck
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=20140404152011.GC5529@katana \
--to=wsa@the-dreams.de \
--cc=agust@denx.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=shawn.guo@linaro.org \
--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: link
Be 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.