From: Andrei Borzenkov <arvidjaar@gmail.com>
To: Gabriele Mazzotta <gabriele.mzt@gmail.com>,
Matthew Garrett <mjg59@srcf.ucam.org>
Cc: "Darren Hart" <dvhart@infradead.org>,
"Pali Rohár" <pali.rohar@gmail.com>,
platform-driver-x86@vger.kernel.org
Subject: Re: Regression with dell-rbtn: radio killed on resume after suspend to RAM
Date: Wed, 25 Nov 2015 06:57:25 +0300 [thread overview]
Message-ID: <565531A5.2030304@gmail.com> (raw)
In-Reply-To: <565430FA.4050301@gmail.com>
24.11.2015 12:42, Gabriele Mazzotta пишет:
> On 24/11/2015 09:52, Matthew Garrett wrote:
>> On Tue, Nov 24, 2015 at 09:49:18AM +0100, Gabriele Mazzotta wrote:
>>>
>>> Pali, what should we do for the case Latitude/Precision + RBTN_TOGGLE?
>>
>> Are there any systems which claim Windows 8 support via OSI and which
>> don't implement the rbtn interface? If not, just add an ACPI dependency
>> to dell-laptop and have it skip the rfkill code on Windows 8 and later
>> systems. I never trusted that code, but Dell insisted that it would
>> always wor on Latitudes and Precisions…
And the code actually worked and still does :)
>>
>
> I think that systems that claim Windows 8 support also implement an
> rbtn interface, but I can't be sure of this.
>
> Such a solution would work, but it's worth pointing out that this would
> move the control of radios from kernel space to user space.
Well, this would be wrong. Button press is still handled by hardware
(BIOS) so we should not pretend it is software. We should rather fix
dell-rbtn to do the right thing when notification is received - i.e.
query current hardware state via dell-laptop.
next prev parent reply other threads:[~2015-11-25 3:57 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-21 18:57 Regression with dell-rbtn: radio killed on resume after suspend to RAM Andrei Borzenkov
2015-11-21 19:08 ` Pali Rohár
2015-11-21 20:39 ` Andrei Borzenkov
2015-11-22 6:23 ` Andrei Borzenkov
2015-11-23 14:50 ` Pali Rohár
2015-11-23 15:14 ` Gabriele Mazzotta
2015-11-23 16:23 ` Andrei Borzenkov
2015-11-23 19:29 ` Darren Hart
2015-11-23 22:33 ` Gabriele Mazzotta
[not found] ` <5653E243.9010707@gmail.com>
2015-11-24 8:32 ` Pali Rohár
2015-11-24 8:49 ` Gabriele Mazzotta
2015-11-24 8:52 ` Matthew Garrett
2015-11-24 9:02 ` Pali Rohár
2015-11-24 9:42 ` Gabriele Mazzotta
2015-11-25 3:57 ` Andrei Borzenkov [this message]
2015-11-25 3:53 ` Andrei Borzenkov
2015-11-25 8:34 ` Andrei Borzenkov
2015-11-25 8:35 ` Pali Rohár
2015-11-25 9:31 ` Gabriele Mazzotta
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=565531A5.2030304@gmail.com \
--to=arvidjaar@gmail.com \
--cc=dvhart@infradead.org \
--cc=gabriele.mzt@gmail.com \
--cc=mjg59@srcf.ucam.org \
--cc=pali.rohar@gmail.com \
--cc=platform-driver-x86@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 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.