ERROR: Macros with complex values should be enclosed in parentheses #53: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:23: +#define TRANS_SEQ_END \ + 0xFFFF, \ + RTW_PWR_CUT_ALL_MSK, \ + RTW_PWR_INTF_ALL_MSK, \ + 0, \ + RTW_PWR_CMD_END, 0, 0 BUT SEE: do {} while (0) advice is over-stated in a few situations: The more obvious case is macros, like MODULE_PARM_DESC, invoked at file-scope, where C disallows code (it must be in functions). See $exceptions if you have one to add by name. More troublesome is declarative macros used at top of new scope, like DECLARE_PER_CPU. These might just compile with a do-while-0 wrapper, but would be incorrect. Most of these are handled by detecting struct,union,etc declaration primitives in $exceptions. Theres also macros called inside an if (block), which "return" an expression. These cannot do-while, and need a ({}) wrapper. Enjoy this qualification while we work to improve our heuristics. WARNING: line length of 81 exceeds 80 columns #185: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:155: + {0x0f, 0x0f, 0x0d, 0x0b, 0x08, 0x05, 0x03, 0x01}, /* 10, -11.0dB */ WARNING: line length of 81 exceeds 80 columns #186: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:156: + {0x10, 0x10, 0x0e, 0x0b, 0x08, 0x05, 0x03, 0x01}, /* 11, -10.5dB */ WARNING: line length of 81 exceeds 80 columns #187: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:157: + {0x11, 0x11, 0x0f, 0x0c, 0x09, 0x06, 0x03, 0x01}, /* 12, -10.0dB */ WARNING: line length of 81 exceeds 80 columns #221: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:191: + {0x0f, 0x0f, 0x0d, 0x08, 0x00, 0x00, 0x00, 0x00}, /* 10, -11.0dB */ WARNING: line length of 81 exceeds 80 columns #222: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:192: + {0x10, 0x10, 0x0e, 0x08, 0x00, 0x00, 0x00, 0x00}, /* 11, -10.5dB */ WARNING: line length of 81 exceeds 80 columns #223: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:193: + {0x11, 0x11, 0x0f, 0x09, 0x00, 0x00, 0x00, 0x00}, /* 12, -10.0dB */ WARNING: line length of 81 exceeds 80 columns #250: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:220: +#define RTW_CCK_SWING_TABLE_SIZE ARRAY_SIZE(rtw8723b_cck_swing_table_ch14) WARNING: line length of 81 exceeds 80 columns #1322: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1292: + struct phy_status_8703b *phy_status = (struct phy_status_8703b *)phy_raw; WARNING: line length of 81 exceeds 80 columns #1340: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1310: + struct phy_status_8703b *phy_status = (struct phy_status_8703b *)phy_raw; WARNING: line length of 82 exceeds 80 columns #1350: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1320: + pkt_stat->rx_snr[RF_PATH_A] = (s8)(phy_status->path_rxsnr[RF_PATH_A] / 2); WARNING: line length of 82 exceeds 80 columns #1551: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1521: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0003f); WARNING: line length of 82 exceeds 80 columns #1552: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1522: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xc7f87); WARNING: line length of 82 exceeds 80 columns #1618: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1588: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #1619: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1589: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xf7fb7); WARNING: line length of 82 exceeds 80 columns #1683: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1653: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #1684: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1654: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xf7d77); WARNING: line length of 81 exceeds 80 columns #1868: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1838: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_ADDR, RFREG_MASK, 0x30000); WARNING: line length of 82 exceeds 80 columns #1869: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1839: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #1870: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1840: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xf7fb7); WARNING: line length of 84 exceeds 80 columns #1963: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1933: + rtw_write32_mask(rtwdev, REG_OFDM0_XAAGC1, MASKBYTE0, 0x50); WARNING: line length of 91 exceeds 80 columns #1964: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1934: + rtw_write32_mask(rtwdev, REG_OFDM0_XAAGC1, MASKBYTE0, backup.igia); WARNING: line length of 81 exceeds 80 columns #1972: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1942: + good = rtw8723x_iqk_similarity_cmp(rtwdev, result, j, i); WARNING: line length of 81 exceeds 80 columns #2018: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1988: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_ADDR, RFREG_MASK, 0x18000); WARNING: line length of 82 exceeds 80 columns #2019: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1989: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #2020: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1990: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xe6177); WARNING: line length of 84 exceeds 80 columns #2107: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2077: + final_ofdm_swing_index - limit_ofdm); WARNING: line length of 88 exceeds 80 columns #2112: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2082: + rtw8723b_pwrtrack_set_ofdm_pwr(rtwdev, path, final_ofdm_swing_index, 0); WARNING: line length of 81 exceeds 80 columns #2116: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2086: + final_cck_swing_index - limit_cck); WARNING: line length of 83 exceeds 80 columns #2247: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2217: + rtw_write8_mask(rtwdev, REG_RFE_CTRL_ANT_SW, BIT_RFE_CTRL_ANT_SW_SEL, 0x3); total: 1 errors, 29 warnings, 0 checks, 2599 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit ce805c775540 ("wifi: rtw88: 8723b: add the RTL8723B chip driver") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE FILE_PATH_CHANGES GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.