All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Arend van Spriel <arend@broadcom.com>
Cc: Linux Wireless List <linux-wireless@vger.kernel.org>,
	Jouni Malinen <j@w1.fi>, Greg Goldman <ggoldman@broadcom.com>,
	Jithu Jance <jithu@broadcom.com>
Subject: Re: [PATCH] nl80211: allow user-space to determine address for P2P_DEVICE
Date: Mon, 21 Jan 2013 11:02:42 +0100	[thread overview]
Message-ID: <1358762562.5190.1.camel@jlt4.sipsolutions.net> (raw)
In-Reply-To: <50FD104F.2090006@broadcom.com>

On Mon, 2013-01-21 at 10:54 +0100, Arend van Spriel wrote:

> >> +	if (type == NL80211_IFTYPE_P2P_DEVICE && info->attrs[NL80211_ATTR_MAC])
> >> +		nla_memcpy(&params.macaddr[0], info->attrs[NL80211_ATTR_MAC],
> >> +			   ETH_ALEN);
> > 
> > It seems like this might need a capability parameter or something, so
> > wpa_s knows this will take effect. OTOH, wpa_s can query the MAC address
> > afterwards, so it's not really needed, so I applied it.
> 
> Not sure if I can follow. I did submit wpa_s patch for flagging the P2P
> device capability. Are you suggesting an additional flag here to
> indicate whether the MAC address can be configured by wpa_s?

Yes, or maybe it's not needed as wpa_s could read the address back after
the interface is created, and see whether it stuck. Right now, for
example, with mac80211 it wouldn't be used at all.

> > I would also like to see a mac80211 change though to make it take effect
> > there.
> 
> See what I can do. I guess it does involve mac80211 device drivers as
> well, right?

Oh, I think there's no mac80211 driver using P2P device yet. :-) I have
a patch for iwlwifi but I'm not merging it until wpa_s gets "more ready"

johannes


      reply	other threads:[~2013-01-21 10:02 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-08  9:17 [PATCH] nl80211: allow user-space to determine address for P2P_DEVICE Arend van Spriel
2013-01-08  9:17 ` [PATCH] iw: additional mac parameter for add interface primitive Arend van Spriel
2013-01-16 22:17 ` [PATCH] nl80211: allow user-space to determine address for P2P_DEVICE Johannes Berg
2013-01-21  9:54   ` Arend van Spriel
2013-01-21 10:02     ` Johannes Berg [this message]

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=1358762562.5190.1.camel@jlt4.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=arend@broadcom.com \
    --cc=ggoldman@broadcom.com \
    --cc=j@w1.fi \
    --cc=jithu@broadcom.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.