linux.git/drivers/staging/rtl8723bs/hal, branch v4.19

staging: rtl8188eu/rtl8723bs: fix character encoding

2018-07-24T11:55:05+00:00

drivers/staging/rtl8188eu/include/odm.h uses an incorrect encoding
for the '...' character in two comments, which makes it one of the
few non-UTF-8 source files.

This removes the odd characters and uses the same ASCII representation
that we have in the regular rtlwifi driver. The second instance
of drivers/staging/rtl8723bs/hal/odm.h is garbled in a different way,
so I change it to be the same as well even though it is already
plain ASCII.

Signed-off-by: Arnd Bergmann 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Add missing curly braces on else statement.

2018-06-28T13:03:16+00:00

Fix 'braces {} should be used on all arms of this statement' coding
style problem in 'phy_get_tx_pwr_lmt()'.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Merge workaround conditionals into single else-if.

2018-06-28T13:03:16+00:00

The if conditionals used to work around wrong TX power limit indices
can be condensed into a single if/else-if statement for more concise
expression.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Merge conditionals with similar bodies.

2018-06-28T13:03:16+00:00

Two conditionals that set 'channel' based on 'band_type' in
'phy_get_tx_pwr_lmt()' can be simplified into one single-line conditional.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Fix spelling mistake in comment.

2018-06-28T13:03:16+00:00

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Move bandwidth index lookup to new function.

2018-06-28T13:03:16+00:00

Factoring out the conditional lookup of bandwidth index into the power
limit table into it's own function simplifies the logic of
'phy_get_tx_pwr_lmt()'.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Move rate section index lookup to new function.

2018-06-28T13:03:16+00:00

The rate section lookup is a large switch statement in the middle of
'phy_get_tx_pwr_lmt()'; refactor this statement into it's own function
for increased readability.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Remove empty else-if conditional.

2018-06-28T13:03:16+00:00

This else-if conditional block does nothing; remove it.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Combine if statements with equivalent body.

2018-06-28T13:03:16+00:00

Two if statements that carry out the same operation can be combined with
a logical OR.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: Fix camel-case names in phy_get_tx_pwr_lmt().

2018-06-28T13:03:16+00:00

Change camel-case names to snake-case names; to avoid variable name
conflicts, rename table index variables to idx_*.

Signed-off-by: Quytelda Kahja 
Signed-off-by: Greg Kroah-Hartman