On 5/24/20 2:26 PM, no-re...@patchew.org wrote: > > This series seems to have some coding style problems. See output below for > more information: > [...] > 3/4 Checking commit 0b01a633fe92 (hw/display/cirrus_vga: Use > qemu_log_mask(ERROR) instead of debug printf) > ERROR: suspect code indent for conditional statements (16, 12) > #31: FILE: hw/display/cirrus_vga.c:1038: > if (s->cirrus_blt_pixelwidth > 2) { > + qemu_log_mask(LOG_GUEST_ERROR,
False positive. Checkpatch is confused by the mis-indented code previous to this line. > > total: 1 errors, 0 warnings, 156 lines checked > > Patch 3/4 has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. [...]