linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Florian Schirmer <jolt@tuxbox.org>
To: Marcelo Tosatti <marcelo.tosatti@cyclades.com>
Cc: obi@saftware.de, carjay@gmx.de,
	linux-ppc-embedded <linuxppc-embedded@ozlabs.org>
Subject: Re: [PATCH] m8xx_wdt: software watchdog reset/interrupt select
Date: Wed, 16 Nov 2005 12:47:31 +0100	[thread overview]
Message-ID: <437B1C53.1060004@tuxbox.org> (raw)
In-Reply-To: <20051115142147.GD32373@logos.cnet>

Hi,

>> The SYPCR register can be set only _once_ at machine startup and the
>> bootloader in question does not have an option to change the mode. Many
>> bootloaders probably dont.

Okay, i was asuming you have control over the bootloader.

> Updated patch addresses code duplication issue you mentioned and also 
> adds an error message in case timer interrupt frequency is higher
> than the watchdog frequency.
> 
> Can I add your Signed-off-by in case you're OK with it?

Maybe i'm missing something obvious, but why is the interrupt driven 
reaming code not working for your configuration. Are you using the PIT 
for something else?

Otherwise i'm fine with the patch. Feel free to add my Signed-off-by line.

Best,
   Florian

  reply	other threads:[~2005-11-16 11:47 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-14 14:38 [PATCH] m8xx_wdt: software watchdog reset/interrupt select Marcelo Tosatti
2005-11-14 20:55 ` Kumar Gala
2005-11-14 15:57   ` [PATCH] m8xx_wdt: software watchdog reset/interrupt select\ Marcelo Tosatti
2005-11-14 16:03     ` Marcelo Tosatti
2005-11-14 21:14       ` Kumar Gala
2005-11-15  6:11       ` Florian Schirmer
2005-11-15  6:41 ` [PATCH] m8xx_wdt: software watchdog reset/interrupt select Florian Schirmer
2005-11-15  5:42   ` Marcelo Tosatti
2005-11-15 14:21     ` Marcelo Tosatti
2005-11-16 11:47       ` Florian Schirmer [this message]
2005-11-16  8:36         ` Marcelo Tosatti
2005-11-17  9:13           ` Florian Schirmer
2005-11-18 15:36             ` Marcelo Tosatti
2005-11-19  6:16               ` Florian Schirmer

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=437B1C53.1060004@tuxbox.org \
    --to=jolt@tuxbox.org \
    --cc=carjay@gmx.de \
    --cc=linuxppc-embedded@ozlabs.org \
    --cc=marcelo.tosatti@cyclades.com \
    --cc=obi@saftware.de \
    /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).