Patrick Georgi
7945f75417
lint/clang-format: set to 96 chars per line
...
coreboot has decided to go with 96 characters per line.
Original Signed-off-by: Patrick Georgi <pgeorgi@google.com >
Original Reviewed-on: https://review.coreboot.org/c/coreboot/+/31651
Change-Id: I9c99e5cca6548e23cf755dc37193ff2aa669ac10
Signed-off-by: Martin Roth <martinroth@google.com >
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33405
Tested-by: build bot (Jenkins) <no-reply@coreboot.org >
Reviewed-by: Philipp Deppenwiese <zaolin.daisuki@gmail.com >
Reviewed-by: Werner Zeh <werner.zeh@siemens.com >
2019-06-13 20:14:00 +00:00
Julius Werner
0e3f7d4780
Revert "lint/clang-format: set to 96 chars per line"
...
This reverts commit 626ba097a2jwerner@chromium.org >
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31916
Tested-by: build bot (Jenkins) <no-reply@coreboot.org >
2019-03-15 23:05:06 +00:00
Patrick Georgi
626ba097a2
lint/clang-format: set to 96 chars per line
...
80 chars + 2 tabs was the compromise we got to in the last round of
discussion.
Change-Id: I9293a69d1bea900da36501cde512004d0695ad37
Signed-off-by: Patrick Georgi <pgeorgi@google.com >
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31651
Tested-by: build bot (Jenkins) <no-reply@coreboot.org >
Reviewed-by: Martin Roth <martinroth@google.com >
Reviewed-by: Nico Huber <nico.h@gmx.de >
2019-03-05 12:45:47 +00:00
Nico Huber
55e3a6ae09
util/lint: Exclude util/inteltool from checkpatch
...
It's causing too much noise during review of register tables.
Change-Id: Iae6cd4454c5ed84b5fe0ea5f8a244e2a2fa13407
Signed-off-by: Nico Huber <nico.huber@secunet.com >
Reviewed-on: https://review.coreboot.org/c/31367
Reviewed-by: Patrick Rudolph <siro@das-labor.org >
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net >
Tested-by: build bot (Jenkins) <no-reply@coreboot.org >
2019-02-20 12:17:32 +00:00
Nico Huber
bfae9a861a
util/lint: Exclude util/superiotool from checkpatch
...
`superiotool` follows its own style (e.g. lot's of missing spaces
and odd placement of braces in the register descriptions).
Change-Id: Ifa33938a0fbac10577cbda10537f856f6f100233
Signed-off-by: Nico Huber <nico.h@gmx.de >
Reviewed-on: https://review.coreboot.org/28214
Reviewed-by: Angel Pons <th3fanbus@gmail.com >
Reviewed-by: Felix Held <felix-coreboot@felixheld.de >
Tested-by: build bot (Jenkins) <no-reply@coreboot.org >
2018-08-20 15:21:50 +00:00
Patrick Georgi
5486786495
checkpatch: exclude util/crossgcc/patches
...
These files are supposed to contain trailing whitespace due to the patch
format. Also use the exclusion list in the pre-commit hook.
Change-Id: I8816c05ea703964a332915a0675096836957b242
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org >
Reviewed-on: https://review.coreboot.org/26695
Reviewed-by: Nico Huber <nico.h@gmx.de >
Reviewed-by: Martin Roth <martinroth@google.com >
Tested-by: build bot (Jenkins) <no-reply@coreboot.org >
2018-05-30 17:12:00 +00:00
Martin Roth
2a063beb71
util/lint: Don't run checkpatch on the documentation
...
Change-Id: Ib95a7c9c64c481af7dcf1074ffc0fc76dc6b6ff9
Signed-off-by: Martin Roth <gaumless@gmail.com >
Reviewed-on: https://review.coreboot.org/19144
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net >
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com >
Reviewed-by: Patrick Georgi <pgeorgi@google.com >
2017-04-24 19:18:06 +02:00
Martin Roth
1ce2ba6832
lint/lint-007-checkpatch: Update lint script
...
- Check Kconfig files as well.
- Accept a list of directories to check as a command line argument.
- Only look at src & util directories by default.
- Skip src/vendorcode.
- Remove bypass of payloads/coreinfo/util/kconfig directory, it no
longer exists.
Change-Id: Ia522d3ddc29914220bdaae36ea23ded7338c48fd
Signed-off-by: Martin Roth <martinroth@google.com >
Reviewed-on: https://review.coreboot.org/16359
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net >
Reviewed-by: Patrick Georgi <pgeorgi@google.com >
2016-09-20 21:43:11 +02:00
Martin Roth
0ecbdde285
lint: rename lint-006-checkpatch because board name is lint-006
...
Checkpatch should be 007.
Change-Id: Ib71c50ad1a63a3a743391cd8fea9f79cd08ef6f3
Signed-off-by: Martin Roth <martinroth@google.com >
Reviewed-on: https://review.coreboot.org/12901
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com >
2016-01-12 22:32:16 +01:00
