lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

regulatory: remove redundant isalpha() check

Browse Source 
toupper() only modifies lower-case letters, so
the isalpha() check is redundant; remove it.

Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
This commit is contained in:
Johannes Berg
2012-12-03 18:59:58 +01:00
parent 11cff96c06
commit d4f2c8819a
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
net/wireless/reg.c
View File

@@ -1614,10 +1614,8 @@ static void reg_todo(struct work_struct *work)
static void queue_regulatory_request(struct regulatory_request *request) static void queue_regulatory_request(struct regulatory_request *request)
{ {
if (isalpha(request->alpha2[0])) request->alpha2[0] = toupper(request->alpha2[0]);
request->alpha2[0] = toupper(request->alpha2[0]); request->alpha2[1] = toupper(request->alpha2[1]);
if (isalpha(request->alpha2[1]))
request->alpha2[1] = toupper(request->alpha2[1]);
spin_lock(&reg_requests_lock); spin_lock(&reg_requests_lock);
list_add_tail(&request->list, &reg_requests_list); list_add_tail(&request->list, &reg_requests_list);