Date: Mon, 18 May 2009 20:03:18 +0300 From: Andriy Gapon <avg@freebsd.org> To: Nate Lawson <nate@root.org> Cc: freebsd-acpi@freebsd.org Subject: Re: 6.x acpi powerbutton Message-ID: <4A1194D6.5020307@freebsd.org> In-Reply-To: <4A118CC1.7060507@freebsd.org> References: <49DE1F8B.2080400@earthlink.net> <49DE2E6D.5050001@icyb.net.ua> <49DE596E.2050406@earthlink.net> <49DEFF53.1040306@icyb.net.ua> <49DF7A1C.90009@root.org> <20090418043432.O34434@sola.nimnet.asn.au> <49E8D824.1000001@root.org> <20090417200726.GG3014@deviant.kiev.zoral.com.ua> <49E8E6E3.40304@root.org> <49EDFAED.4030606@icyb.net.ua> <49EE1863.1000306@root.org> <49EF15B0.10402@icyb.net.ua> <4A11870D.1030805@freebsd.org> <4A118CC1.7060507@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
on 18/05/2009 19:28 Andriy Gapon said the following: > on 18/05/2009 19:04 Andriy Gapon said the following: >> Following up on this - maybe we can just change ACPI_VPRINT to device_printf in >> acpi_button_notify_sleep? > > Oh, nonsense, this would work only for "control method" button case. > So printf needs to be in acpi_event_power_button_sleep and > acpi_event_sleep_button_sleep... > BTW, from the code in acpi_button ("ACPI_FPB", "ACPI_FSB") it seems that the original intention was to use that driver for both fixed and "control method" buttons, but in fact the fixed case is handled directly in acpi.c code. -- Andriy Gapon
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4A1194D6.5020307>