mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Ian Campbell <icampbell@arcom.com>
To: Wim Van Sebroeck <wim@iguana.be>
Cc: Russell King <linux@arm.linux.org.uk>,
	Nicolas Pitre <nico@cam.org>,
	linux-kernel@vger.kernel.org
Subject: deactivating PXA255 watchdog
Date: Thu, 02 Jun 2005 09:18:04 +0100	[thread overview]
Message-ID: <1117700284.3063.51.camel@icampbell-debian> (raw)

Hi Wim,

I think you are the current "watchdog guy", although watchdog doesn't
have a MAINTAINERS entry so I'm looking at bk/lkml history etc and might
be mistaken...

I wrote to the the linux-arm-kernel mailing list recently with a query
because the sa1100_wdt code supports the writing 'V' to deactivate the
watchdog thing, but unfortunately once armed the hardware cannot be
deactivated. 

I was going to produce a patch to remove the 'V' support and it was
suggested by Russell that I should ask for your opinion. Do you have a
preference with regards to offering a warning or error etc if a 'V' is
written?

My original message is below. I have had confirmation that the SA1100
has the same behaviour as PXA2xx from Nico.

Ian.

> The code in drivers/char/watchdog/sa1100_wdt.c has support for the
> standard write a 'V' for a safe shutdown semantics, which clears the
> OIER[E3] bit.
> 
> However, it seems that even if this bit is clear the watchdog reset will
> still occur because OWER[WME] is set (and cannot be cleared).
> 
> Removing this support (perhaps with a warning/error or something) would
> seem to make sense but I wanted to check that the SA1100 behaviour was
> the same since they share the driver before I submitted anything.

-- 
Ian Campbell, Senior Design Engineer
                                        Web: http://www.arcom.com
Arcom, Clifton Road,                    Direct: +44 (0)1223 403 465
Cambridge CB1 7EA, United Kingdom       Phone:  +44 (0)1223 411 200


             reply	other threads:[~2005-06-02  8:20 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-02  8:18 Ian Campbell [this message]
2005-08-09 19:24 ` Wim Van Sebroeck
2005-08-10  7:57   ` Ian Campbell

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=1117700284.3063.51.camel@icampbell-debian \
    --to=icampbell@arcom.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nico@cam.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®