From: Andrew Morton <akpm@linux-foundation.org>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: linux-input@vger.kernel.org, linux-acpi@vger.kernel.org,
linux-kernel@vger.kernel.org, kristen.c.accardi@intel.com
Subject: Re: RESEND: [PATCH 2/3] ACPI: Send switch event on dock events
Date: Wed, 28 May 2008 22:32:25 -0700 [thread overview]
Message-ID: <20080528223225.51cf8298.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080528132152.GB32646@srcf.ucam.org>
On Wed, 28 May 2008 14:33:49 +0100 Matthew Garrett <mjg59@srcf.ucam.org> wrote:
> Send a switch event on docking for consistency with docks that don't
> present as a separate device.
>
> Signed-off-by: Matthew Garrett <mjg@redhat.com>
>
> ---
>
> diff --git a/drivers/acpi/dock.c b/drivers/acpi/dock.c
> index fa44fb9..a67e74c 100644
> --- a/drivers/acpi/dock.c
> +++ b/drivers/acpi/dock.c
> @@ -25,6 +25,7 @@
> #include <linux/kernel.h>
> #include <linux/module.h>
> #include <linux/init.h>
> +#include <linux/input.h>
> #include <linux/types.h>
> #include <linux/notifier.h>
> #include <linux/platform_device.h>
> @@ -65,6 +66,7 @@ struct dock_station {
> struct mutex hp_lock;
> struct list_head dependent_devices;
> struct list_head hotplug_devices;
> + struct input_dev *dock_input;
> };
>
> struct dock_dependent_device {
> @@ -345,10 +347,13 @@ static void dock_event(struct dock_station *ds, u32 event, int num)
> char event_string[13];
> char *envp[] = { event_string, NULL };
>
> - if (num == UNDOCK_EVENT)
> + if (num == UNDOCK_EVENT) {
> + input_report_switch(dock_station->dock_input, SW_DOCK, 0);
Is CONFIG_ACPI_DOCK=y, CONFIG_INPUT=n possible? I think so..
next prev parent reply other threads:[~2008-05-29 5:32 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-23 16:09 [PATCH] Add switch for dock events Matthew Garrett
2008-05-24 1:23 ` Andrew Morton
2008-05-28 13:06 ` [PATCH 1/3] Input: " Matthew Garrett
2008-05-28 13:21 ` [PATCH 2/3] ACPI: Send switch event on " Matthew Garrett
2008-05-28 13:29 ` [PATCH 3/3] Misc: Add HP WMI laptop extras driver Matthew Garrett
2008-05-29 5:42 ` RESEND: " Andrew Morton
2008-05-29 8:07 ` Matthew Garrett
2008-05-29 8:23 ` Matthew Garrett
2008-06-02 23:30 ` Andrew Morton
2008-06-07 10:00 ` Carlos Corbacho
2008-05-29 5:32 ` Andrew Morton [this message]
2008-05-29 8:15 ` RESEND: [PATCH 2/3] ACPI: Send switch event on dock events Matthew Garrett
2008-05-29 8:22 ` Version 2: " Matthew Garrett
2008-06-09 9:48 ` Andrew Morton
2008-05-29 8:20 ` Version 2: [PATCH 1/3] Input: Add switch for " Matthew Garrett
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080528223225.51cf8298.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=kristen.c.accardi@intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).