[PATCH v3 0/3] Resolve checkpatch if/else brace style errors

Shobhit Kukreti shobhitkukreti at gmail.com
Sun Jun 16 16:19:48 UTC 2019

This patchset fixes the following errors reported by checkpatch in the 
staging/rtl8723bs driver.

Patch[1/3]: Fix check patch error "that open brace { should be on the previous line"

Patch[2/3]: Fix the error else should follow close brace '}' 

Patch[3/3]: Fix Indentation Error

version 3 changes:
	- Converted the patch to a patchset
	- Resolve checkpatch errors:
		else should follow  close brace '}'
		Fixed Indentation Error to use tabs
	- Compiles and builds, untested on real hardware.

version 2 changes:
	- Removed Trailing whitespace introduced in the previous patch
        - Moved comments to a new line in the else statement

Shobhit Kukreti (3):
  staging: rtl8723bs: Resolve checkpatch error "that open brace { should
    be on the previous line" in the rtl8723 driver
  staging: rtl8723bs: Resolve the checkpatch error: else should follow  
      close brace '}'
  staging: rtl8723bs: Fix Indentation Error: code indent should use tabs
        where possible

 drivers/staging/rtl8723bs/os_dep/mlme_linux.c     | 19 ++---
 drivers/staging/rtl8723bs/os_dep/recv_linux.c     | 90 ++++++++---------------
 drivers/staging/rtl8723bs/os_dep/rtw_proc.c       |  6 +-
 drivers/staging/rtl8723bs/os_dep/sdio_intf.c      | 54 ++++++--------
 drivers/staging/rtl8723bs/os_dep/sdio_ops_linux.c | 24 ++----
 5 files changed, 71 insertions(+), 122 deletions(-)


More information about the devel mailing list