Re: IB700 watchdog

2016-03-19 Thread Mike Larkin
On Fri, Mar 18, 2016 at 08:18:30PM +0100, Mark Kettenis wrote: > > Date: Thu, 17 Mar 2016 14:28:04 +0900 > > From: Masao Uebayashi > > > > This tiny driver is only meant for a backup watchdog configuration on > > QEMU, whose ICH watchdog, supported by ichwdt(4), seems

Re: IB700 watchdog

2016-03-19 Thread Martin Pieuchot
On 17/03/16(Thu) 14:28, Masao Uebayashi wrote: > This tiny driver is only meant for a backup watchdog configuration on > QEMU, whose ICH watchdog, supported by ichwdt(4), seems broken. What's the problem? Is it in our code or in QEMU?

Re: IB700 watchdog

2016-03-18 Thread Mark Kettenis
> Date: Thu, 17 Mar 2016 14:28:04 +0900 > From: Masao Uebayashi > > This tiny driver is only meant for a backup watchdog configuration on > QEMU, whose ICH watchdog, supported by ichwdt(4), seems broken. > > ib700wdt(4) supports only 0-30 second timeouts but in practice

Re: IB700 watchdog

2016-03-18 Thread Theo de Raadt
+#ib700wdt* at isa? port 0x441 # iBase 700 (IB700) Watchdog Timer We don't do drivers with numbers in their names. +int +ib700wdt_match(struct device *parent, void *match, void *aux) +{ + struct isa_attach_args *ia = aux; + + ia->ia_iosize = IB700WDT_SIZE; + ia->ia_msize = 0;