On Mon, 2014-01-27 at 12:21 +0200, Ilan Peer wrote:
> switch (user_reg_hint_type) {
> case NL80211_USER_REG_HINT_USER:
> case NL80211_USER_REG_HINT_CELL_BASE:
> - break;
> + if (!info->attrs[NL80211_ATTR_REG_ALPHA2])
> + return -EINVAL;
> +
> + data = nla_data(info->attrs[NL80211_ATTR_REG_ALPHA2]);
> + return regulatory_hint_user(data, user_reg_hint_type);
> + case NL80211_USER_REG_HINT_INDOOR:
> + return regulatory_hint_indoor_user();
> default:
> return -EINVAL;
> }
>
> - r = regulatory_hint_user(data, user_reg_hint_type);
> -
> - return r;
> + return 0;
Isn't that "return 0" dead code now?
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
