Re: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <thomas.mingarelli@...>
Cc: <linux-kernel@...>, <thomas.mingarelli@...>
Date: Monday, October 15, 2007 - 2:15 pm

On Mon, 15 Oct 2007 14:05:50 -0400 (EDT)
thomas.mingarelli@hp.com wrote:



Hi,

Your patch looks quite clean in general; however it does make me wonder
if it should either leverage or expand the arch/x86/pci/pcbios.c
infrastructure for doing BIOS32 calls and share that.... it's kinda
unpleasant as a general thought to have drivers poke this deep into
various guts of the system/bios.... esp if the common code has to do
something very similar already.

Greetings,
   Arjan van de Ven
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch, Andrew Morton, (Mon Oct 22, 4:51 am)
Re: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch, Arjan van de Ven, (Mon Oct 15, 2:15 pm)
RE: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch, Mingarelli, Thomas, (Mon Oct 15, 4:24 pm)