All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alan Jenkins <alan-jenkins@tuffmail.co.uk>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	linux-wireless@vger.kernel.org
Subject: Re: rfkill, stupid question #6
Date: Mon, 03 Nov 2008 18:00:36 +0000	[thread overview]
Message-ID: <490F3C44.7080504@tuffmail.co.uk> (raw)
In-Reply-To: <20081103163311.GA2417@khazad-dum.debian.net>

Henrique de Moraes Holschuh wrote:
> On Mon, 03 Nov 2008, Matthew Garrett wrote:
>   
>> On Mon, Nov 03, 2008 at 01:02:29PM -0200, Henrique de Moraes Holschuh wrote:
>>     
>>> Right now, you should still rfkill_force_state().  Please wait for an hour
>>> or two while I clean up that broken resume handling, and I will tell you for
>>> sure.
>>>       
>> Cool. I'll hold off posting my cleanups until then in that case.
>>     
>
> Ok, two bugs reproduced, the fixes are ready and tested, and I will be
> sending it now to linux-wireless.  You're in the CC, so you will get them.
>
> I will also need to send patches for -stable, as the ones for mainline won't
> apply to -stable.
>
> Now, for what you asked: you DO NOT HAVE to use rfkill_force_state() in your
> driver's resume method, as long as you NEVER make use of
> RFKILL_STATE_HARD_BLOCKED.   I have fixed the bug that was messing this up.
>   

Thanks for fixing this, even though it doesn't affect my
non-STATE_HARD_BLOCKED-using hardware.

I have one more question.  I read that if a STATE_SOFT_BLOCKED request
is made when the hardware is in STATE_HARD_BLOCKED, the rfkill driver is
expected to "double block".

If the hard block is later cleared, the driver is expected to call
rfkill_force_state(SOFT_BLOCKED).  The SOFT_BLOCKED state can then be
cleared as normal.

But if there is an UNBLOCK request in the double-blocked state, the
rfkill core will reject it and preserve the  double-blocked state.  Is
this intended, or a known issue?

Wouldn't it be simpler to use a bitmask so that the rfkill core can at
least represent this double-blocked state?  I guess the problem would be
how to shoehorn it into the sysfs interface.

Thanks
Alan

  reply	other threads:[~2008-11-03 18:00 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-31 17:09 eeepc-laptop rfkill, stupid question #4 and 5 Alan Jenkins
2008-10-31 17:11 ` Matthew Garrett
2008-10-31 17:27   ` Alan Jenkins
2008-10-31 20:54     ` Alan Jenkins
2008-11-02  4:00       ` Henrique de Moraes Holschuh
2008-11-02 11:17         ` eeepc-laptop rfkill, stupid question #4 Alan Jenkins
2008-11-02 13:06           ` Matthew Garrett
2008-11-02 13:25             ` Alan Jenkins
2008-11-02 13:26               ` Matthew Garrett
2008-11-03 14:47               ` Henrique de Moraes Holschuh
2008-11-03 14:16             ` Henrique de Moraes Holschuh
2008-11-03 14:18               ` Matthew Garrett
2008-11-03 14:29                 ` Alan Jenkins
2008-11-03 14:51                   ` Henrique de Moraes Holschuh
2008-11-03 14:55                     ` Matthew Garrett
2008-11-03 15:02                       ` Henrique de Moraes Holschuh
2008-11-03 15:08                         ` Matthew Garrett
2008-11-03 16:33                           ` Henrique de Moraes Holschuh
2008-11-03 18:00                             ` Alan Jenkins [this message]
2008-11-03 19:06                               ` rfkill, stupid question #6 Henrique de Moraes Holschuh
2008-11-04 15:48                             ` eeepc-laptop rfkill, stupid question #4 Luiz Fernando N. Capitulino
2008-11-04 15:57                               ` Alan Jenkins
2008-11-02  3:46   ` eeepc-laptop rfkill, stupid question #4 and 5 Henrique de Moraes Holschuh
2008-11-02  9:21     ` Matthew Garrett

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=490F3C44.7080504@tuffmail.co.uk \
    --to=alan-jenkins@tuffmail.co.uk \
    --cc=hmh@hmh.eng.br \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@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.