Subject: Re: Bug in dev/hpc/hpcapm.c
To: None <port-hpcsh@netbsd.org>
From: Valeriy E. Ushakov <uwe@ptc.spbu.ru>
List: port-hpcsh
Date: 08/22/2006 05:55:45
On Tue, Aug 22, 2006 at 03:11:55 +0200, Adam Wysocki via ArcaBit wrote:

> http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/dev/hpc/hpcapm.c?rev=1.2.4.1

Why not HEAD? (just curious)


> It contains:
> 
>         sc->sc_battery_hook = config_hook(CONFIG_HOOK_PMEVENT,
>                                           CONFIG_HOOK_PMEVENT_BATTERY,
>                                           CONFIG_HOOK_SHARE,
>                                           hpcapm_hook, sc);
> 
>         sc->sc_battery_hook = config_hook(CONFIG_HOOK_PMEVENT,
>                                           CONFIG_HOOK_PMEVENT_AC,
>                                           CONFIG_HOOK_SHARE,
>                                           hpcapm_hook, sc);
> 
> Shouldn't the latter be replaced with sc->sc_ac_hook?

It should.  Fixed.  Thanks.

SY, Uwe
-- 
uwe@ptc.spbu.ru                         |       Zu Grunde kommen
http://snark.ptc.spbu.ru/~uwe/          |       Ist zu Grunde gehen