qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Fahlgren <daniel@fahlgren.se>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org, "Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] watchdog: add support to emulate winbond w83627thf
Date: Fri, 16 Oct 2015 16:46:53 +0200	[thread overview]
Message-ID: <1445006813.3872.22.camel@fahlgren-laptop> (raw)
In-Reply-To: <5614E149.40000@redhat.com>

Hi Paolo,

On Wed, 2015-10-07 at 11:09 +0200, Paolo Bonzini wrote:
> 
> On 07/10/2015 10:49, Daniel Fahlgren wrote:
> > Hi,
> > 
> > On Mon, 2015-09-14 at 22:07 +0200, Daniel Fahlgren wrote:
> >> This patch adds support to emulate the watchdog functionality on the
> >> Winbond w83627thf chip. The other features of the chip are not emulated.
> >> It has been tested with Ubuntu 6.06, 14.04 and 15.04 as guests using the
> >> w83627hf_wdt module.
> > 
> > Ping, who should I poke about this? The maintainers file does not
> > mention the watchdog system.
> 
> Hi,
> 
> sorry for missing this patch.
> 
> I have a couple of questions.  First, where can I find a spec for this
> chip in order to review the code, and what are the other features?
> Second, what are the advantages over the existing watchdog devices?

The spec can be found at
http://www.alldatasheet.com/datasheet-pdf/pdf/206865/WINBOND/W83627THF.html

To quote the datasheet about the other features:

"83627THF is a Winbond LPC I/O product. It integrates the following
major peripheral functions in a chip: the disk driver adapter (FDC),
Serial port (UART), Parallel port (SPP/EPP/ECP), Keyboard controller
(KBC), SIR, Game port, MIDI port, Hardware Monitor, ACPI, On Now Wake-Up
features."

My reason for implementing this watchdog is that I have a Linux system
image that depends on /dev/watchdog being present. That image only has a
single kernel module for watchdogs enabled which happened to be this
one. So no advantages except diversity and giving the users the option
to choose from different devices.

Best regards,
Daniel Fahlgren

  reply	other threads:[~2015-10-16 14:47 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-14 20:07 [Qemu-devel] [PATCH] watchdog: add support to emulate winbond w83627thf Daniel Fahlgren
2015-10-07  8:49 ` Daniel Fahlgren
2015-10-07  9:09   ` Paolo Bonzini
2015-10-16 14:46     ` Daniel Fahlgren [this message]
2015-10-16 15:11       ` Paolo Bonzini

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=1445006813.3872.22.camel@fahlgren-laptop \
    --to=daniel@fahlgren.se \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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;
as well as URLs for NNTP newsgroup(s).