From b8d0d089b307a8e40f12af82d0191d52f8752b37 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Tue, 14 Jul 2020 18:21:43 +0200 Subject: Documentation: Remove mention of manual checkpatch.pl invocation We typically call checkpatch.pl through make lint which properly adds the option to tell checkpatch that our lines may be 96 columns long. However there's one mention of calling checkpatch directly and that confuses people with complaints about overly long lines that exceed 80 columns. The lint test that runs checkpatch (and with the right options) can also be used on a per-directory basis, so offer that instead. Change-Id: If21e925d2d2394c876724a44b0e23c9b2744c56b Signed-off-by: Patrick Georgi Reviewed-on: https://review.coreboot.org/c/coreboot/+/43450 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Angel Pons Reviewed-by: Nico Huber --- Documentation/tutorial/part2.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'Documentation') diff --git a/Documentation/tutorial/part2.md b/Documentation/tutorial/part2.md index 5ab210a591..4ac857473d 100644 --- a/Documentation/tutorial/part2.md +++ b/Documentation/tutorial/part2.md @@ -108,8 +108,7 @@ in the source files. To see errors that are already present, build the files in the repository by running `make lint` in the coreboot directory. Alternatively, if you want to run `make lint` on a specific directory, run: - for file in $(git ls-files | grep ); do \ - util/lint/checkpatch.pl --file $file --terse; done + util/lint/lint-007-checkpatch where `filepath` is the filepath of the directory (ex. `src/cpu/amd/car`). -- cgit v1.2.3