On Mon, 2014-04-21 at 12:53 +0800, Zhao, Gang wrote: > Change "err" to "ret", since return value isn't zero doesn't always > mean error occured, sometimes it means success. > > if (err) > /* do things if function successfully returned */ > > looks strange to me, so change it. > > This change related to the functions which call > cfg80211_chandef_dfs_check(), since cfg80211_chandef_dfs_check() > returns 1 means dfs is required. Same here, there's a lot of work ongoing with CSA around the code, I don't want to disrupt all 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