On 01/14/2014 08:11 PM, Johannes Berg wrote: > On Tue, 2014-01-14 at 15:41 +0800, Ying Xue wrote: > >> @@ -2218,10 +2194,6 @@ static int nl80211_set_wiphy(struct sk_buff *skb, >> struct genl_info *info) >> rdev->wiphy.coverage_class = old_coverage_class; >> } >> } >> - >> - bad_res: >> - if (netdev) >> - dev_put(netdev); >> return result; > > I believe that could be "return 0;" now, which would make the code > easier to read. >
Yes, I will change it in next version. Thanks, Ying > johannes > > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
