CHECK: Please use a blank line after function/struct/union/enum declarations #125: FILE: tools/testing/selftests/net/can/test_raw_filter.c:112: +}; +FIXTURE_VARIANT_ADD(can_filters, base_eff) { CHECK: Please use a blank line after function/struct/union/enum declarations #132: FILE: tools/testing/selftests/net/can/test_raw_filter.c:119: +}; +FIXTURE_VARIANT_ADD(can_filters, base_rtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #139: FILE: tools/testing/selftests/net/can/test_raw_filter.c:126: +}; +FIXTURE_VARIANT_ADD(can_filters, base_effrtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #154: FILE: tools/testing/selftests/net/can/test_raw_filter.c:141: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_eff_eff) { CHECK: Please use a blank line after function/struct/union/enum declarations #161: FILE: tools/testing/selftests/net/can/test_raw_filter.c:148: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_eff_rtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #168: FILE: tools/testing/selftests/net/can/test_raw_filter.c:155: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_eff_effrtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #183: FILE: tools/testing/selftests/net/can/test_raw_filter.c:170: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_rtr_eff) { CHECK: Please use a blank line after function/struct/union/enum declarations #190: FILE: tools/testing/selftests/net/can/test_raw_filter.c:177: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_rtr_rtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #197: FILE: tools/testing/selftests/net/can/test_raw_filter.c:184: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_rtr_effrtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #212: FILE: tools/testing/selftests/net/can/test_raw_filter.c:199: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_effrtr_eff) { CHECK: Please use a blank line after function/struct/union/enum declarations #219: FILE: tools/testing/selftests/net/can/test_raw_filter.c:206: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_effrtr_rtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #226: FILE: tools/testing/selftests/net/can/test_raw_filter.c:213: +}; +FIXTURE_VARIANT_ADD(can_filters, filter_effrtr_effrtr) { CHECK: Please use a blank line after function/struct/union/enum declarations #241: FILE: tools/testing/selftests/net/can/test_raw_filter.c:228: +}; +FIXTURE_VARIANT_ADD(can_filters, eff_eff) { CHECK: Alignment should match open parenthesis #269: FILE: tools/testing/selftests/net/can/test_raw_filter.c:253: + TH_LOG("filters: can_id = 0x%08X can_mask = 0x%08X", + rfilter.can_id, rfilter.can_mask); WARNING: line length of 81 exceeds 80 columns #304: FILE: tools/testing/selftests/net/can/test_raw_filter.c:275: + ret = read(self->sock, &frame, sizeof(struct can_frame)); WARNING: line length of 82 exceeds 80 columns #331: FILE: tools/testing/selftests/net/can/test_raw_filter.c:277: + TH_LOG("failed to read frame %d (%d)", rx, errno); WARNING: line length of 106 exceeds 80 columns #339: FILE: tools/testing/selftests/net/can/test_raw_filter.c:285: + rxbitval = 1 << ((frame.can_id & (CAN_EFF_FLAG|CAN_RTR_FLAG|CAN_ERR_FLAG)) >> 28); CHECK: spaces preferred around that '|' (ctx:VxV) #339: FILE: tools/testing/selftests/net/can/test_raw_filter.c:285: + rxbitval = 1 << ((frame.can_id & (CAN_EFF_FLAG|CAN_RTR_FLAG|CAN_ERR_FLAG)) >> 28); ^ CHECK: spaces preferred around that '|' (ctx:VxV) #339: FILE: tools/testing/selftests/net/can/test_raw_filter.c:285: + rxbitval = 1 << ((frame.can_id & (CAN_EFF_FLAG|CAN_RTR_FLAG|CAN_ERR_FLAG)) >> 28); ^ total: 0 errors, 3 warnings, 16 checks, 349 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 42432e9bccec ("selftests: can: Use fixtures in test_raw_filter") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE 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. total: 0 errors, 3 warnings, 16 checks, 349 lines checked