All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Dan Williams <dcbw@redhat.com>
Cc: linux-wireless@vger.kernel.org
Subject: Re: [RFT 3/4] cfg80211: add rfkill support
Date: Sun, 24 May 2009 19:24:11 +0200	[thread overview]
Message-ID: <1243185851.29222.4.camel@johannes.local> (raw)
In-Reply-To: <1243171957.24236.13.camel@localhost.localdomain>

[-- Attachment #1: Type: text/plain, Size: 1326 bytes --]

On Sun, 2009-05-24 at 09:32 -0400, Dan Williams wrote:
> On Thu, 2009-05-21 at 23:59 +0200, Johannes Berg wrote:
> > plain text document attachment (015-cfg80211-rfkill.patch)
> > To be easier on drivers and users, have cfg80211 register an
> > rfkill structure that drivers can access. When soft-killed,
> > simply take down all interfaces; when hard-killed the driver
> > needs to notify us and we will take down the interfaces
> > after the fact. While rfkilled, interfaces cannot be set UP.
> 
> Any chance we could get something other than EINVAL returned from the
> PRE_UP notifier when the device is rfkilled?  It's useful to get some
> indication of *why* the device can't be brought up.  For example, if the
> device requires firmware but that firmware isn't present, the driver
> will usually return ENOENT because that's what request_firmware()
> returns when it can't find what it needs.  Thus userspace can do
> something intelligent with the error.
> 
> So maybe EL2HLT could be (ab)used here?  I guess technically it's an L1
> halt since the PHY is powered down, but whatever.  The only other stuff
> that seems to use this are the USB 'gadget' drivers.  Either that, or
> lets make an EL1OFF or ERFKILL maybe.

Sure, we can do whatever we want, I just quickly put EINVAL in.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

  reply	other threads:[~2009-05-24 17:25 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-21 21:59 [RFT 0/4] rfkill improvements Johannes Berg
2009-05-21 21:59 ` [RFT 1/4] net: introduce pre-up netdev notifier Johannes Berg
2009-05-21 21:59 ` [RFT 2/4] rfkill: add function to query state Johannes Berg
2009-05-21 21:59 ` [RFT 3/4] cfg80211: add rfkill support Johannes Berg
2009-05-23  9:52   ` [RFT 3/4 v2] " Johannes Berg
2009-05-24 13:32   ` [RFT 3/4] " Dan Williams
2009-05-24 17:24     ` Johannes Berg [this message]
2009-05-26 16:38     ` Marcel Holtmann
2009-05-21 21:59 ` [RFT 4/4] iwlwifi: port to cfg80211 rfkill Johannes Berg
2009-05-23  8:49   ` [RFT 4/4 v2] " Johannes Berg
2009-05-23  8:52 ` [RFT 5/4] iwm: port to new " Johannes Berg
2009-05-25  1:22   ` Zhu Yi
2009-05-25  5:13     ` Johannes Berg

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=1243185851.29222.4.camel@johannes.local \
    --to=johannes@sipsolutions.net \
    --cc=dcbw@redhat.com \
    --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.