On Fri, 2011-04-01 at 10:35 +0200, Roland Vossen wrote:
> Code cleanup. Use Linux function instead of Broadcom specific
> one. On spots where the device object is not yet available,
> pr_err() is used instead.
Perhaps use wiphy_<level> when possible.
> diff --git a/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c
> b/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c
[]
> @@ -205,8 +207,8 @@ wl_ops_add_interface(struct ieee80211_hw *hw, struct
> ieee80211_vif *vif)
> vif->type != NL80211_IFTYPE_STATION &&
> vif->type != NL80211_IFTYPE_WDS &&
> vif->type != NL80211_IFTYPE_ADHOC) {
> - WL_ERROR("%s: Attempt to add type %d, only STA for now\n",
> - __func__, vif->type);
> + dev_err(&hw->wiphy->dev, "%s: Attempt to add type %d, only"
> + " STA for now\n", __func__, vif->type);
wiphy_err(hw->wiphy, "%s: Attempt to add type %d, only STA for
now\n",
__func__, vif->type);
etc...
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
