staging: rtl8712: Use completions for signaling
authorSathish Kumar <skumark1902@gmail.com>
Wed, 23 Mar 2022 04:55:15 +0000 (10:25 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 4 Apr 2022 05:33:45 +0000 (07:33 +0200)
commita3515f20535fc6a3813da45268911a65f5ebe362
tree6438af8d091ed3092e99010e2ac5a2d38995174f
parent52a0af7091b41deb8cd27d7954b43d30ab27f3ab
staging: rtl8712: Use completions for signaling

r8712_sitesurvey_cmd() uses a variable to notify r8712_SetFilter() that it
has completed operation. There is no sort of assurance that the variable will
actually change and it could cache the value the first time it is read and
then never update it for the whole loop logic.

Use completion variables because they are better suited for the purpose.

This patch fixes the checkpatch.pl warnings like:
CHECK: Avoid CamelCase: <blnEnableRxFF0Filter>
+   u8 blnEnableRxFF0Filter;

Reviewed-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
Signed-off-by: Sathish Kumar <skumark1902@gmail.com>
Link: https://lore.kernel.org/r/20220323045515.2513-1-skumark1902@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8712/drv_types.h
drivers/staging/rtl8712/rtl871x_cmd.c
drivers/staging/rtl8712/usb_intf.c
drivers/staging/rtl8712/xmit_linux.c