These patches fix three categories of checkpatch warnings (redundant braces, redundant else branches after return/break, and constant placements on tests) in file rtw_mlme_ext.c, which is part of the rtl8723bs driver. False positives, such as if (foo) { /* TODO: ... */ } were not altered. --- v1 -> v2: Removing a redundant else branch left a variable declaration mixed with code, which was caught by the kernel test bot. Move the declaration to the beginning of a function. --- Sevinj Aghayeva (3): staging: rtl8723bs: remove redundant braces in if statements staging: rtl8723bs: place constants on the right side of tests staging: rtl8723bs: remove redundant else branches drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 167 ++++++++---------- 1 file changed, 70 insertions(+), 97 deletions(-) -- 2.25.1