wil6210: match wait_for_completion_timeout return type
authorNicholas Mc Guire <hofrat@osadl.org>
Thu, 13 Aug 2015 12:38:31 +0000 (15:38 +0300)
committerKalle Valo <kvalo@qca.qualcomm.com>
Mon, 17 Aug 2015 14:38:55 +0000 (17:38 +0300)
Return type of wait_for_completion_timeout is unsigned long not int.
As remain is exclusively used for wait_for_completion_timeout here its
type is simply changed to unsigned long.

API conformance testing for completions with coccinelle spatches are being
used to locate API usage inconsistencies:
./drivers/net/wireless/ath/wil6210/wmi.c:827
int return assigned to unsigned long

Patch was compile tested with x86_64_defconfig + CONFIG_ATH_CARDS=m,
CONFIG_WIL6210=m

Patch is against 4.1-rc3 (localversion-next is -next-20150514)

Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/wil6210/wmi.c

index 7a25736..2f35d4c 100644 (file)
@@ -806,7 +806,7 @@ int wmi_call(struct wil6210_priv *wil, u16 cmdid, void *buf, u16 len,
             u16 reply_id, void *reply, u8 reply_size, int to_msec)
 {
        int rc;
-       int remain;
+       unsigned long remain;
 
        mutex_lock(&wil->wmi_mutex);