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