Re: [PATCH] ACPI: thinkpad-acpi: always enable input layer, add hotkey_report_mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Henrique,
I may be seriously jet-lagged right now,
but I'm having trouble parsing this check-in comment
and the the Documentation update, so I'm not
clear on what you're trying to do with this code.

It should basically say:

hotkeys are reported through the input layer,
and /proc/acpi/event when CONFIG_ACPI_PROC_EVENT=y

all other events are reported via netlink,
and /proc/acpi/event when CONFIG_ACPI_PROC_EVENT=y

And if you need to add a flag to eliminate duplicate
reporting, so be it.

But I'm totally confused why there is any mention of netlink
in any of this text -- for if this patch is really
specific to hotkey events, then no version of any
kernel should send hotkey events via netlink,
and no version of HAL should listen for hotkey
events on netlink.

thanks,
-Len

ps.
For me to apply a patch post -rc5,
it really has to be a bug fix -- preferably fixing a regression.
ie. "revert new 2.6.23 CONFIG_THINKPAD_ACPI_INPUT_ENABLED option
because it would create a legacy we don't want to support"
would be a suitable synopsis -- "always enable input layer"
sort of gives the impression that this is a new feature, not a fix...

pps. 
On Sunday 02 September 2007 23:16, Henrique de Moraes Holschuh wrote:
> Now that ACPICA can send events over netlink, we can use a different

Note that ACPICA doesn't know anything about how events are sent
to user-space -- is it Linux specific ACPI code that does this.
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux