Re: [PATCH] dell-wmi Switch support and Tablet switch definition

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

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Matthew Garrett wrote:
> On Fri, Jun 19, 2009 at 03:28:48PM -0400, Rafi Rubin wrote:
> 
>> I agree polling the tablet state would be a good thing.  But that does not mean we should not
>> support the signals until we are ready to support polling.
>>
>> If you are suggesting that we should not use an EV_SW because we will eventually poll the hardware
>> instead of keeping the state in software, then I have no problem with sending keys or other events.
> 
> No, my point is that there's little benefit in providing EV_SW if the 
> initial value is going to be wrong. Software simply can't depend on it. 
> The code's absolutely fine, but there's a risk that it'll break 
> otherwise working desktops until we know how to set the initial value.


The signal is not a toggle.  Its an explicit "going to tablet" and "coming from tablet".  And the
initial value is "non-tablet" mode, which is what must be assumed without this support.

So the only new case where a wrong assumption may be made is if the screen is flipped out of tablet
mode during suspend or hibernate.  But we _don't_ have the potential of a toggle being inverted and
it rotating the screen to something silly every time you flip it.

I think the benefits of the partial support far outweight the potential hazards.


That being said, if you have any clue how to probe the state, I would be very happy to help you.
And I'd agree that signals can wait.


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAko79b8ACgkQwuRiAT9o60+VowCg5zvB16xzUz6w7cqP6twu4xKv
9o0AoJuG3v9qkiB8SXVpk/D9nud2A8FM
=h/y8
-----END PGP SIGNATURE-----
--
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