linux.git/drivers/staging/rtl8723bs/hal, branch v5.1

Staging: rtl8723bs: Fix code style issue in rtl8723bs driver

2018-12-17T13:26:10+00:00

Add space between 'else' and '{' to solve code style issue
in all the files for rtl8723bs driver.

Signed-off-by: Hardik Singh Rathore 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: change semaphores to completions

2018-12-12T10:50:46+00:00

This driver uses many semaphores, most of them are equivalent to
completions. The other copies of this driver got moved over to
completions a while ago, so do the same here.

In this usage scenario, the two are equivalent, so the behavior
should not change.

Signed-off-by: Arnd Bergmann 
Reviewed-by: Hans de Goede 
Signed-off-by: Greg Kroah-Hartman

Merge 4.20-rc5 into staging-next

2018-12-03T06:55:28+00:00

We need the staging fixes in here as well.

Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Correct errors from checkpatch

2018-11-20T09:53:54+00:00

Correct following errors reported by checkpath.pl:

ERROR: space required before the open parenthesis '(' #265: FILE:
drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c:265:
+                       if(!precvframe) ')'

Also similar errors on line 274 and 283.

Signed-off-by: Josenivaldo Benito Jr 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Fix incorrect sense of ether_addr_equal

2018-11-07T12:08:28+00:00

In commit b37f9e1c3801 ("staging: rtl8723bs: Fix lines too long in
update_recvframe_attrib()."), the refactoring involved replacing
two memcmp() calls with ether_addr_equal() calls. What the author
missed is that memcmp() returns false when the two strings are equal,
whereas ether_addr_equal() returns true when the two addresses are
equal. One side effect of this error is that the strength of an
unassociated AP was much stronger than the same AP after association.
This bug is reported at bko#201611.

Fixes: b37f9e1c3801 ("staging: rtl8723bs: Fix lines too long in update_recvframe_attrib().")
Cc: Stable 
Cc: youling257 
Cc: u.srikant.patnaik@gmail.com
Reported-and-tested-by: youling257 
Signed-off-by: Larry Finger 
Signed-off-by: Greg Kroah-Hartman

drivers: staging: rtl*: fix spelling mistake "Orginial" -> "Original"

2018-11-07T11:35:10+00:00

Trivial fix to common spelling mistakes in some rtl* drivers

Signed-off-by: Colin Ian King 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: fix spelling mistake "Abnrormal" -> "Abnormal"

2018-10-14T11:29:34+00:00

Trivial fix to spelling mistake in message text

Signed-off-by: Colin Ian King 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Remove unnecessary parentheses and dead commented code

2018-10-09T13:13:24+00:00

Clang warns when multiple pairs of parentheses are used for a single
conditional statement.

drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:3351:20: warning:
equality comparison with extraneous parentheses [-Wparentheses-equality]

The commented code is pointless since _HW_STATE_AP_ is handled right
below this block.

Link: https://github.com/ClangBuiltLinux/linux/issues/168
Signed-off-by: Nathan Chancellor 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: clean up clang warning on extraneous parentheses

2018-10-02T21:59:14+00:00

There are extraneous parantheses that are causing clang to produce a
warning so remove these.

Clean up 3 clang warnings:
equality comparison with extraneous parentheses [-Wparentheses-equality]

Signed-off-by: Colin Ian King 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: fix spelling mistake "unvalid" -> "invalid"

2018-09-25T19:02:06+00:00

Trivial fix to spelling mistake in DBG_871X debug messages.

Signed-off-by: Colin Ian King 
Signed-off-by: Greg Kroah-Hartman