Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34386 )
Change subject: util/lint/checkpatch.pl: Update line length to 96 ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/34386/1/util/lint/checkpatch.pl File util/lint/checkpatch.pl:
https://review.coreboot.org/c/coreboot/+/34386/1/util/lint/checkpatch.pl@54 PS1, Line 54: my $max_line_length = 96;
An issue with the old 80-char limit on checkpatch still being present was brought up on the leadersh […]
Hmmm... maybe I was wrong. It looks like we do still get the warning sometimes (e.g. https://review.coreboot.org/c/coreboot/+/33387/9/src/mainboard/emulation/qem... ). I could swear that I've seen violations that weren't pointed out elsewhere, but maybe I'm confused (or maybe that was in file types checkpatch doesn't cover or something).
Either way, then we should figure out why the way it's currently done doesn't work.