ERROR: Macros with complex values should be enclosed in parentheses #58: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:24: +#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 #203: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:169: + {0x0f, 0x0f, 0x0d, 0x0b, 0x08, 0x05, 0x03, 0x01}, /* 10, -11.0dB */ WARNING: line length of 81 exceeds 80 columns #204: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:170: + {0x10, 0x10, 0x0e, 0x0b, 0x08, 0x05, 0x03, 0x01}, /* 11, -10.5dB */ WARNING: line length of 81 exceeds 80 columns #205: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:171: + {0x11, 0x11, 0x0f, 0x0c, 0x09, 0x06, 0x03, 0x01}, /* 12, -10.0dB */ WARNING: line length of 81 exceeds 80 columns #239: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:205: + {0x0f, 0x0f, 0x0d, 0x08, 0x00, 0x00, 0x00, 0x00}, /* 10, -11.0dB */ WARNING: line length of 81 exceeds 80 columns #240: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:206: + {0x10, 0x10, 0x0e, 0x08, 0x00, 0x00, 0x00, 0x00}, /* 11, -10.5dB */ WARNING: line length of 81 exceeds 80 columns #241: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:207: + {0x11, 0x11, 0x0f, 0x09, 0x00, 0x00, 0x00, 0x00}, /* 12, -10.0dB */ WARNING: line length of 81 exceeds 80 columns #268: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:234: +#define RTW_CCK_SWING_TABLE_SIZE ARRAY_SIZE(rtw8723b_cck_swing_table_ch14) WARNING: line length of 83 exceeds 80 columns #910: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:876: + swing = rtw_read32_mask(rtwdev, REG_OFDM_0_XA_TX_IQ_IMBALANCE, 0xffc00000); WARNING: line length of 98 exceeds 80 columns #1294: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1260: + rtw_write_rf(rtwdev, RF_PATH_A, RF_AC, MASK12BITS, (rf_mode & 0x8ffff) | 0x10000); WARNING: line length of 81 exceeds 80 columns #1590: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1556: + struct phy_status_8703b *phy_status = (struct phy_status_8703b *)phy_raw; WARNING: line length of 81 exceeds 80 columns #1608: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1574: + struct phy_status_8703b *phy_status = (struct phy_status_8703b *)phy_raw; WARNING: line length of 82 exceeds 80 columns #1618: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1584: + pkt_stat->rx_snr[RF_PATH_A] = (s8)(phy_status->path_rxsnr[RF_PATH_A] / 2); WARNING: line length of 82 exceeds 80 columns #1820: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1786: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0003f); WARNING: line length of 82 exceeds 80 columns #1821: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1787: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xc7f87); WARNING: line length of 82 exceeds 80 columns #1890: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1856: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #1891: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1857: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xf7fb7); WARNING: line length of 82 exceeds 80 columns #1957: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1923: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #1958: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:1924: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xf7d77); WARNING: line length of 81 exceeds 80 columns #2146: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2112: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_ADDR, RFREG_MASK, 0x30000); WARNING: line length of 82 exceeds 80 columns #2147: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2113: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #2148: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2114: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xf7fb7); WARNING: line length of 84 exceeds 80 columns #2238: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2204: + rtw_write32_mask(rtwdev, REG_OFDM0_XAAGC1, MASKBYTE0, 0x50); WARNING: line length of 91 exceeds 80 columns #2239: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2205: + rtw_write32_mask(rtwdev, REG_OFDM0_XAAGC1, MASKBYTE0, backup.igia); WARNING: line length of 81 exceeds 80 columns #2252: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2218: + good = rtw8723x_iqk_similarity_cmp(rtwdev, result, j, i); WARNING: line length of 81 exceeds 80 columns #2298: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2264: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_ADDR, RFREG_MASK, 0x18000); WARNING: line length of 82 exceeds 80 columns #2299: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2265: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA0, RFREG_MASK, 0x0001f); WARNING: line length of 82 exceeds 80 columns #2300: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2266: + rtw_write_rf(rtwdev, RF_PATH_A, RF_MODE_TABLE_DATA1, RFREG_MASK, 0xe6177); WARNING: line length of 84 exceeds 80 columns #2379: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2345: + final_ofdm_swing_index - limit_ofdm); WARNING: line length of 88 exceeds 80 columns #2384: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2350: + rtw8723b_pwrtrack_set_ofdm_pwr(rtwdev, path, final_ofdm_swing_index, 0); WARNING: line length of 81 exceeds 80 columns #2388: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2354: + final_cck_swing_index - limit_cck); WARNING: line length of 83 exceeds 80 columns #2545: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2511: + rtw_write8_mask(rtwdev, REG_RFE_CTRL_ANT_SW, BIT_RFE_CTRL_ANT_SW_SEL, 0x3); WARNING: line length of 81 exceeds 80 columns #2878: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2844: + * so the firmware-driven rate adaptation WARNING: line length of 81 exceeds 80 columns #2881: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2847: + * legacy 8051 8723b/8703b/8723d firmware WARNING: line length of 82 exceeds 80 columns #2883: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2849: + * older 8821a/8812a chips. byte4..bw fall WARNING: line length of 81 exceeds 80 columns #2885: FILE: drivers/net/wireless/realtek/rtw88/rtw8723b.c:2851: + * rtw_fw_ra_report_iter(), which matches total: 1 errors, 35 warnings, 0 checks, 2950 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 e6dbc687ee98 ("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.