WARNING: 'timout' may be misspelled - perhaps 'timeout'? #48: FILE: src/lib/agent.c:89: + /* set default tick timout */ CHECK: braces {} should be used on all arms of this statement #148: FILE: src/lib/work.c:188: + if (reschedule_usecs == 0) [...] - atomic_sub(&work_shared_ctx->total_num_work, [...] CHECK: Unbalanced braces around else statement #157: FILE: src/lib/work.c:190: + else { CHECK: multiple assignments should be avoided #181: FILE: src/lib/work.c:227: + ticks = current = work_get_timer(queue); CHECK: Blank lines aren't necessary after an open brace '{' #185: FILE: src/lib/work.c:231: + list_for_item(wlist, &queue->work) { + ERROR: Missing Signed-off-by: line(s) total: 1 errors, 1 warnings, 4 checks, 328 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. 0001-Revert-work-shared-work-queue-implementation.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.