All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Mattia Dongili <malattia@linux.it>
Cc: linux-acpi@vger.kernel.org
Subject: Re: [PATCH] sony-laptop: support rfkill via ACPI interfaces
Date: Sat, 21 Mar 2009 04:35:13 +0000	[thread overview]
Message-ID: <20090321043513.GA20519@srcf.ucam.org> (raw)
In-Reply-To: <20090321040010.GD3834@kamineko.org>

On Sat, Mar 21, 2009 at 01:00:10PM +0900, Mattia Dongili wrote:
> On Fri, Mar 20, 2009 at 02:00:04PM +0000, Matthew Garrett wrote:
> > I had one machine where ECON seemed to need to be called explicitly, but 
> > I can't remember the details now. Calling it probably wouldn't hurt 
> > anything.
> 
> seems to be a TT and Z specific thing though. The DSDT on other models
> doesn't provide the ECON method.

Yeah. As I said, I don't think there's any harm in causing it - I think 
I was getting more promising results from hotkey events in the Z when I 
called ECON, but I don't have access to that machine right now and never 
got it finished off.

> > The numbers correspond to enabling all events. I couldn't think of any 
> > reason why we'd only want to enable a subset. The current nc setup code 
> > seems to enable some events and then disable them again, which I don't 
> > really understand.
> 
> Well, the current sequence was taken from a trace in windows on a Vaio C
> Type, then it demonstrated to be helpful on other models as well.
> The SN07[1] method is very different from the Z and TT type to the AR, C,
> FE, FZ and N so I'm starting to suspect that we're just seeing a new
> generation of SNC based models. I'll see if some users with older models
> can give the new sequence a go.

Looking through, the implementation seems quite different but the 
functionality seems the same - the newer machines seem to return values 
directly, whereas older ones tended to trap into SMM. The wireless 
control (at least, the enumeration call I make) seems to be a noop on 
these older machines. It /looks/ like we can probably get some sort of 
versioning information about the interface by calling SN00. I think that 
would probably be a better approach than using DMI for this.

I've put this into rawhide, so I suspect we'll hear complaints if it 
breaks things for anybody.
-- 
Matthew Garrett | mjg59@srcf.ucam.org

  reply	other threads:[~2009-03-21  4:35 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-19 21:21 [PATCH] sony-laptop: support rfkill via ACPI interfaces Matthew Garrett
2009-03-19 21:28 ` Matthew Garrett
2009-03-19 21:34 ` Norbert Preining
2009-03-19 21:44   ` Matthew Garrett
2009-03-19 21:49     ` Norbert Preining
2009-03-19 21:56       ` Matthew Garrett
2009-03-19 22:15     ` Norbert Preining
2009-03-20  0:28       ` Norbert Preining
2009-03-20  0:38         ` Matthew Garrett
2009-03-20  0:40           ` Norbert Preining
2009-03-20  1:18             ` Norbert Preining
2009-03-20  7:33               ` Matthias Welwarsky
2009-03-21 11:22               ` Matthias Welwarsky
2009-03-21 13:53                 ` Matthias Welwarsky
2009-03-21 14:45                   ` Mattia Dongili
2009-03-21 16:51                     ` Norbert Preining
2009-03-22 17:56                   ` Matthew Garrett
2009-03-22 18:03                     ` Matthew Garrett
2009-03-22 20:36                       ` Norbert Preining
2009-03-22 20:37                         ` Matthew Garrett
2009-03-22 22:06                           ` Norbert Preining
2009-03-22 22:46                             ` Matthew Garrett
2009-03-22 23:10                               ` Mattia Dongili
2009-03-22 23:14                                 ` Matthew Garrett
2009-03-23  0:08                                   ` Mattia Dongili
2009-03-23  0:10                                     ` Matthew Garrett
2009-03-23 12:30                                 ` Norbert Preining
2009-03-23 13:04                                   ` Mattia Dongili
2009-03-23 15:32                                     ` Norbert Preining
2009-03-23 15:43                                       ` Matthew Garrett
2009-03-23 16:00                                         ` Norbert Preining
2009-03-23 16:09                                           ` Matthew Garrett
2009-03-23 16:27                                             ` Norbert Preining
2009-03-23 16:30                                               ` Matthew Garrett
2009-03-23 16:37                                                 ` Norbert Preining
2009-03-23 16:40                                                   ` Matthew Garrett
2009-03-23 16:41                                                     ` Norbert Preining
2009-03-23 16:51                                                       ` Matthew Garrett
2009-03-23 17:48                                                         ` Norbert Preining
2009-03-23 19:51                                                           ` Matthew Garrett
2009-03-24  0:01                                                             ` Norbert Preining
2009-03-24  0:08                                                             ` Mattia Dongili
2009-03-23 21:48                                 ` Matthew Garrett
2009-03-24  0:02                                   ` Norbert Preining
2009-03-24  0:04                                     ` Matthew Garrett
2009-03-23 12:29                               ` Norbert Preining
2009-03-23 14:58                                 ` Matthew Garrett
2009-03-21 16:18                 ` Norbert Preining
2009-03-20  8:52 ` Mattia Dongili
2009-03-20 14:00   ` Matthew Garrett
2009-03-21  4:00     ` Mattia Dongili
2009-03-21  4:35       ` Matthew Garrett [this message]
2009-03-21  6:32         ` Mattia Dongili
2009-03-21 14:06           ` Matthew Garrett
2009-03-21 14:37             ` Mattia Dongili
2009-03-21 14:55               ` Matthew Garrett
2009-03-21 15:10                 ` Matthew Garrett
2009-03-21 19:15                   ` Matthias Welwarsky
2009-03-22 13:33                     ` Matthew Garrett
2009-03-22  2:38                   ` Mattia Dongili

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=20090321043513.GA20519@srcf.ucam.org \
    --to=mjg59@srcf.ucam.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=malattia@linux.it \
    /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.