Re: [PATCH] WDT Driver for Book-E [2/2] Device driver part.

On Fri, Mar 04, 2005 at 10:02:40AM -0600, Kumar Gala wrote:

Matt,

Do you think this really needs a config option?  Should it not be  
sufficient that if you enable watchdog you get this functionality on  
4xx/e500?

I see the 8xx example, however, there may be a case where somebody
wants to use another hardware watchdog mechanism.  In that case,
they probably don't want this facility automagically being enabled.
I think having the option can't hurt.

-Matt

Fair enough, we might actually have multiple watchdogs on some of are=20
parts when the integrate system logic w/its own watchdog.  Makes sense.=20=

 Lets leave it a config option then.

- kumar

On Mar 4, 2005, at 10:35 AM, Matt Porter wrote:

On Fri, Mar 04, 2005 at 10:02:40AM -0600, Kumar Gala wrote:
 > Matt,
 >

quoted

Do you think this really needs a config option?=A0 Should it not be=A0=

quoted

sufficient that if you enable watchdog you get this functionality =

on=A0

quoted

4xx/e500?

I see the 8xx example, however, there may be a case where somebody
 wants to use another hardware watchdog mechanism.=A0 In that case,
 they probably don't want this facility automagically being enabled.
I think having the option can't hurt.

-Matt