On Fri, 2012-11-16 at 22:47 -0800, Marco Porsch wrote:
> +/**
> + * ieee80211_has_qos_pm - check Power Save Level in QoS control
> + * @qc - QoS control bytes in little-endian byteorder
> + */
> +static inline int ieee80211_has_qos_pm(__le16 qc)
bool
> +/**
> + * ieee80211_set_peer_ps_mode - track the neighbor mesh STA's peer-specific
> + * power mode towards the local STA
not valid kernel-doc.
> +void ieee80211_set_nonpeer_ps_mode(struct sta_info *sta,
> + struct ieee80211_hdr *hdr)
> +{
> + enum nl80211_mesh_power_mode pm;
> + static const char *modes[] = {
> + [NL80211_MESH_POWER_ACTIVE] = "active",
> + [NL80211_MESH_POWER_DEEP_SLEEP] = "deep sleep",
> + };
> + mps_dbg(sta->sdata, "STA %pM sets non-peer mode to %s\n",
> + sta->sta.addr, modes[pm]);
This will cause warnings if debug is disabled, can't you just print the
numeric value? It's for your own debugging -- hardly seems necessary to
have the strings.
Same in the other function.
And now that I've seen you haven't even compiled the code with the debug
Kconfig turned off, I'll have to ask you to do that.
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
