| Message ID | 20200828020542.393022-1-ruscur@russell.cc (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 0fb4871bcc8997acbb8edf14b301fc150101d6c0 |
| Headers | show |
| Series | powerpc/tools: Remove 90 line limit in checkpatch script | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (d4ecce4dcc8f8820286cf4e0859850c555e89854) |
| snowpatch_ozlabs/build-ppc64le | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/build-ppc64be | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/build-ppc64e | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/build-pmac32 | warning | Upstream build failed, couldn't test patch |
| snowpatch_ozlabs/checkpatch | warning | total: 1 errors, 0 warnings, 0 checks, 7 lines checked |
| snowpatch_ozlabs/needsstable | success | Patch has no Fixes tags |
On Fri, 28 Aug 2020 12:05:42 +1000, Russell Currey wrote: > As of commit bdc48fa11e46, scripts/checkpatch.pl now has a default line > length warning of 100 characters. The powerpc wrapper script was using > a length of 90 instead of 80 in order to make checkpatch less > restrictive, but now it's making it more restrictive instead. > > I think it makes sense to just use the default value now. Applied to powerpc/next. [1/1] powerpc/tools: Remove 90 line limit in checkpatch script https://git.kernel.org/powerpc/c/0fb4871bcc8997acbb8edf14b301fc150101d6c0 cheers
diff --git a/arch/powerpc/tools/checkpatch.sh b/arch/powerpc/tools/checkpatch.sh index 3ce5c093b19d..91c04802ec31 100755 --- a/arch/powerpc/tools/checkpatch.sh +++ b/arch/powerpc/tools/checkpatch.sh @@ -9,7 +9,6 @@ script_base=$(realpath $(dirname $0)) exec $script_base/../../../scripts/checkpatch.pl \ --subjective \ --no-summary \ - --max-line-length=90 \ --show-types \ --ignore ARCH_INCLUDE_LINUX \ --ignore BIT_MACRO \
As of commit bdc48fa11e46, scripts/checkpatch.pl now has a default line length warning of 100 characters. The powerpc wrapper script was using a length of 90 instead of 80 in order to make checkpatch less restrictive, but now it's making it more restrictive instead. I think it makes sense to just use the default value now. Signed-off-by: Russell Currey <ruscur@russell.cc> --- arch/powerpc/tools/checkpatch.sh | 1 - 1 file changed, 1 deletion(-)