Re: [PATCH 00/15] hpwdt: make NMI code a config option (+ other cleanup)

dann frazier
2010-08-13T07:29:29Z

On Wed, Aug 11, 2010 at 02:49:11PM +0200, Wim Van Sebroeck wrote:
> Hi Dan,
>
> > > watchdog: hpwdt (10/12): Construct status message w/ kasprintf and emit it with dev_info
> > > watchdog: hpwdt (11/12): Use "decoding" instead of "sourcing"
> > > watchdog: hpwdt (12/12): Make NMI decoding a compile-time option
> > >
> > > (patch 12 can still be improved).
> >
> > Thanks! I'll work on the #ifdef'ing & get you a new #12
>
> I did some changes in the last 3 patches:
> I removed patch 10, made patch 11 patch 10 and added and extra patch that moves the nmi decoding init and exit functions in seperate modules.
> I'll sent you the 3 new patches. Just let me know if they are OK.

Yes, they look very much like the changes I was preparing - but I got
sidetracked by vacation before I could test :) I pulled your tree &
ran a few sanity tests - looks good. Thanks for doing this!

--
dann frazier | ProLiant Linux