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