Dear coreboot folks,
Each commit pushed to Gerrit is automatically tested for “formal” issues by using `checkpatch.pl`. See for example [1]. Though despite missing a space violating our coding style, which is also found by `checkpatch.pl` [2], the comment contains, that no errors is found. > https://qa.coreboot.org/job/coreboot-checkpatch/5142/ : SUCCESS ``` ERROR: space required before the open brace '{' #49: FILE: src/mainboard/google/gru/pwm_regulator.c:61: + } else if (IS_ENABLED(CONFIG_BOARD_GOOGLE_KEVIN) && board_id() >= 6){ total: 1 errors, 0 warnings, 49 lines checked ``` Is there a reason for not relaying these errors? If not, it’d be great to do so (and for the Chromium and Intel folks to also do that in their repository). Thanks, Paul [1] https://review.coreboot.org/18460 [2] https://qa.coreboot.org/job/coreboot-checkpatch/5142/console
signature.asc
Description: This is a digitally signed message part
-- coreboot mailing list: coreboot@coreboot.org https://www.coreboot.org/mailman/listinfo/coreboot