public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Vroon <tony@linx.net>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: Ivo van Doorn <ivdoorn@gmail.com>,
	linux-acpi@vger.kernel.org, jwoithe@physics.adelaide.edu.au
Subject: Re: Fuijtsu Lifebook RFKILL support
Date: Thu, 11 Dec 2008 17:33:47 +0000	[thread overview]
Message-ID: <1229016827.6446.6.camel@localhost> (raw)
In-Reply-To: <20081211165247.GA4844@khazad-dum.debian.net>

[-- Attachment #1: Type: text/plain, Size: 1013 bytes --]

On Thu, 2008-12-11 at 14:52 -0200, Henrique de Moraes Holschuh wrote:
> That is an input device, that should issue EV_SW SW_RFKILL_ALL events (and
> not EV_KEY, etc).

Okay, glad I asked.
Can I combine EV_SW & EV_KEY on a single input device (as there is
already a FUJ02E3 input device within the device driver) or would that
break things?

Also, would there be a clear acknowledgement of an rfkill state change
in Gnome?

> So, you won't even #include <linux/rfkill.h>.

Noted, include removed. 

> The only pitfall I know of is that you have to re-sync the input layer at every point it could have missed
> a state change.  This means you must issue gratuitous EV_SW SW_RFKILL_ALL
> events with the current state of the switch after you created the input
> device, after resume (from S3/S4), and when you get the notifier from ACPI
> that the switch state could have changed.

Done at input device creation and when notified. I have no resume
callback to hook into.

Regards,
Tony V.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

  reply	other threads:[~2008-12-11 17:34 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-11  1:05 Fuijtsu Lifebook RFKILL support Tony Vroon
2008-12-11  1:20 ` Jonathan Woithe
2008-12-11  1:31   ` Tony Vroon
2008-12-11  1:44     ` Jonathan Woithe
2008-12-11 16:52 ` Henrique de Moraes Holschuh
2008-12-11 17:33   ` Tony Vroon [this message]
2008-12-11 19:47     ` Henrique de Moraes Holschuh
2008-12-12  1:50       ` Tony Vroon
2008-12-12 19:53         ` Henrique de Moraes Holschuh
2008-12-12 20:33           ` Len Brown
2008-12-13 12:47             ` Tony Vroon
2008-12-13 13:28               ` Henrique de Moraes Holschuh
2008-12-13 17:57                 ` Matthew Garrett
2008-12-13 20:55                   ` Tony Vroon
2008-12-13 21:24                     ` Matthew Garrett
2008-12-14  3:13                   ` Henrique de Moraes Holschuh
2008-12-14 17:05                 ` Dan Williams
2008-12-15 11:53                   ` Henrique de Moraes Holschuh
     [not found]                     ` <20081215115336.GD17648-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org>
2008-12-15 15:19                       ` Dan Williams
2008-12-15 17:14                         ` Tony Vroon
2008-12-15 17:59                           ` Dan Williams
2008-12-16 13:50                             ` Tony Vroon
2008-12-16 15:23                               ` Dan Williams
2008-12-15 23:42                           ` Jonathan Woithe
2008-12-15 23:48                             ` Tony Vroon
2008-12-16  0:02                               ` Jonathan Woithe

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=1229016827.6446.6.camel@localhost \
    --to=tony@linx.net \
    --cc=hmh@hmh.eng.br \
    --cc=ivdoorn@gmail.com \
    --cc=jwoithe@physics.adelaide.edu.au \
    --cc=linux-acpi@vger.kernel.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