Re: [PATCH, new ACPI driver] new sony_acpi driver

Vojtech Pavlik
2005-02-14T20:31:42Z

On Mon, Feb 14, 2005 at 11:58:37AM +0100, Stelian Pop wrote:

> > Stelian Pop <stelian@popies.net> wrote:
> >
> > > Privately I've had many positive feedbacks from users of this driver
> > > (and no negative feedback), including Linux distributions who wish
> > > to include it into their kernels. The reports are increasing in number,
> > > it would seem that newer Sony Vaios are more and more incompatible
> > > with sonypi and require sony_acpi to control the screen brightness.
> >
> > The sonypi driver seems to be necessary to catch Vaio hotkey events,
> > including the sleep button. I've checked a couple of DSDTs, and it seems
> > that the more recent Vaios are lacking the SPIC entries but still don't
> > have the sleep button defined. Is there any chance of this driver being
> > able to catch hotkey events?
>
> I don't believe so.
>
> > Related to that, I have a nastyish hack which lets the sonypi driver
> > generate ACPI events whenever a hotkey is pressed. Despite not strictly
> > being ACPI events, this makes it much easier to integrate sonypi stuff
> > with general ACPI support. I'll send it if you're interested.
>
> Wouldn't be more useful to make the ACPI hotkeys generate an
> input event (like sonypi does) and integrate all this at the input
> level ?

Yes, I'd like to see that. The other possible way is have the input
layer generate ACPI events for power-related keys.

--
Vojtech Pavlik
SuSE Labs, SuSE CR