On Wed, 2013-04-24 at 10:13 +0200, Arend van Spriel wrote:
> If android is moving to 3.8 kernel with KLP (key lime pie), you could
> say that it requires middleware/wifi-framework to deal with the new
> kernel API. However, it mostly involves wpa_supplicant regardless of
> using it on android or not. So this is more a wpa_supplicant P2P hack.
There's no driver in 3.8 though that even uses P2P-Device. :-)
> >>> +#ifdef CONFIG_CFG80211_ANDROID_P2P_HACK
> >>> + || (wdev->iftype == NL80211_IFTYPE_P2P_DEVICE &&
> >>> + info->genlhdr->cmd != NL80211_CMD_SET_INTERFACE)
> >>> +#endif
> >>> + ) {
> >>> mutex_unlock(&cfg80211_mutex);
> >>> if (rtnl)
> >>> rtnl_unlock();
> >>
> >> And doing an #ifdef in the middle of if-clause doesn't ring any alarm
> >> bells that this might be a really bad idea ;)
> >
> > Well, I could put an #ifdef into each and every command instead which I
> > didn't want to, which incidentally is the reason for not allowing this
> > with WEXT, WEXT ioctls on the P2P-Device netdev just cause crashes :)
>
> I dislike these kind of constructs as well so I would like this to be
> made somewhat cleaner (conditional helper function?).
Actually I had to change it anyway, see the v2 patch, to fix a crash
when removing the p2p-device again.
johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
