Re: [PATCH] sony-laptop: support rfkill via ACPI interfaces

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

 



On Sat, Mar 21, 2009 at 08:15:00PM +0100, Matthias Welwarsky wrote:
> On Saturday 21 March 2009 16:10:37 Matthew Garrett wrote:
> > The way that 0x113 moves around leaves me pretty sure that calling SN07
> > without checking what the function signature is is not a good idea. I'll
> > rewrite my patch.
> 
> If that's really a function signature, why are some of the slots empty? If 
> you're supposed to read the CFGI first via SN00, that would not make much 
> sense. However, it makes sense if the function is encoded by the slot and a 
> hole (i.e. Zero) means that the function is not supported or disabled.

I thought that at first, but 0x113 is the same code on the machines I 
checked even though it moves around. 0x100 appears to be "Get hotkey 
event" on the newer hardware even though it's not in slot 2.

> If you're right though, it means that "0x124" on all new-style models (those 
> with SN07 method) must always operate the killswitch. I might be able to find 
> that out from the decompiled DSDT. Right now I only checked the Z series DSDTs 
> (Z11 and Z21) and of course they are very similar. But I would be very 
> interested to see the code of  functions called for the other "slot 3" 
> methods, like 0x0104 and 0x0113. If you had some DSDTs to share...

The P, TT and Z all seem to have 0x124 as killswitch - the 0x113 and 
0x104 methods don't seem to be killswitch related, as far as I can tell. 
Mattia linked to a set of DSDTs earlier in the thread.

-- 
Matthew Garrett | mjg59@xxxxxxxxxxxxx
--
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