All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carlos Corbacho <cathectic@gmail.com>
To: "Éric Piel" <Eric.Piel@tremplin-utc.net>
Cc: Dmitry Torokhov <dtor@insightbb.com>,
	linux-input@atrey.karlin.mff.cuni.cz,
	Ashutosh Naik <ashutosh.naik@gmail.com>
Subject: Re: wistron_btns - Is polling always required?
Date: Fri, 27 Jul 2007 12:20:50 +0100	[thread overview]
Message-ID: <200707271220.51397.cathectic@gmail.com> (raw)
In-Reply-To: <46A7C375.4080109@tremplin-utc.net>

On Wednesday 25 July 2007 22:41:09 Éric Piel wrote:
> So I'm fine with your suggestion to remove their entries, at least for:
> 3020, 5020, 2100, 2410. For the 2420, it came earlier from Ashutosh Naik
> (cc'ed now), asking him first if it's also working without the driver
> would be much better!

Fair enough. :-)

> By removing the entries, we should keep in mind that with the vanilla
> kernel, there is still a small functionality loss: wireless and led
> activations are lost. 

True, but then again, this still really shouldn't be done by polling (perhaps 
some sort of userspace application would do-the-right-thing?)

> Those functionalities are provided by the 
> acer_acpi driver (with a much cleaner approach), 

For the 3020/ 5020 yes. For the other machines I do not have DSDTs for them 
yet, so I don't know if acer_acpi will work with them or not (unfortunately 
acer_acpi is limited to a small subset of Acer's 2005 and newer laptops - I 
just don't know all of them yet).

> but it's unlikely to 
> reach the vanilla kernel within less than three or four releases. 

Possibly - it won't go in as is though (I'm currently waiting on Len Brown to 
release a generic WMI ACPI implementation that acer_acpi can then build on, 
and hopefully _that_ will then get submitted upstream; acer_acpi in it's 
current form has been NAKed).

> So 
> Dimitry should decide if we want to only remove the key part from those
> entries, leaving the led and wireless part until acer_acpi gets merged
> or if we should delete them completely right now.

I would say not to base this decision on acer_acpi - I can't yet guarantee it 
will support anything other than the 3020/5020 of the other listed laptops. I 
would say at the very least remove the 5020 from wistron_btns, otherwise it's 
left with inconsistent support between x86 and x86-64.

-Carlos
-- 
E-Mail: cathectic@gmail.com
Web: strangeworlds.co.uk
GPG Key ID: 0x23EE722D

  reply	other threads:[~2007-07-27 11:20 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-19 17:03 wistron_btns - Is polling always required? Carlos Corbacho
2007-07-20  4:06 ` Dmitry Torokhov
2007-07-22 14:14   ` Carlos Corbacho
2007-07-25 21:41     ` Éric Piel
2007-07-27 11:20       ` Carlos Corbacho [this message]
2007-07-27 13:07         ` Éric Piel
2007-07-27 13:27           ` Carlos Corbacho

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=200707271220.51397.cathectic@gmail.com \
    --to=cathectic@gmail.com \
    --cc=Eric.Piel@tremplin-utc.net \
    --cc=ashutosh.naik@gmail.com \
    --cc=dtor@insightbb.com \
    --cc=linux-input@atrey.karlin.mff.cuni.cz \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.