[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/18] ppc-for-5.1 queue 20200507
From: |
no-reply |
Subject: |
Re: [PULL 00/18] ppc-for-5.1 queue 20200507 |
Date: |
Wed, 6 May 2020 23:00:40 -0700 (PDT) |
Patchew URL: https://patchew.org/QEMU/address@hidden/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: address@hidden
Subject: [PULL 00/18] ppc-for-5.1 queue 20200507
Type: series
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
Switched to a new branch 'test'
2d7cf2f target-ppc: fix rlwimi, rlwinm, rlwnm for Clang-9
8a40c99 spapr_nvdimm: Tweak error messages
c711884 spapr_nvdimm.c: make 'label-size' mandatory
3b966ed target/ppc: Add support for Radix partition-scoped translation
841c68d target/ppc: Rework ppc_radix64_walk_tree() for partition-scoped
translation
a6648da target/ppc: Extend ppc_radix64_check_prot() with a 'partition_scoped'
bool
ba64285 target/ppc: Introduce ppc_radix64_xlate() for Radix tree translation
12ffc72 spapr: Don't allow unplug of NVLink2 devices
b807de2 target/ppc: Assert if HV mode is set when running under a pseries
machine
b921172 target/ppc: Introduce a relocation bool in
ppc_radix64_handle_mmu_fault()
0f1ea37 target/ppc: Enforce that the root page directory size must be at least 5
0265feb spapr: Drop CAS reboot flag
660bb29 spapr/cas: Separate CAS handling from rebuilding the FDT
ae65da6 spapr: Simplify selection of radix/hash during CAS
d9869ed ppc/pnv: Add support for NMI interface
f0ed65a ppc/spapr: tweak change system reset helper
d8a29c1 spapr: Don't check capabilities removed between CAS calls
a1a2597 target/ppc: Improve syscall exception logging
=== OUTPUT BEGIN ===
1/18 Checking commit a1a2597979cf (target/ppc: Improve syscall exception
logging)
ERROR: code indent should never use tabs
#42: FILE: target/ppc/excp_helper.c:73:
+^I^I " r4=%016" PRIx64 " r5=%016" PRIx64 " r6=%016" PRIx64$
ERROR: code indent should never use tabs
#43: FILE: target/ppc/excp_helper.c:74:
+^I^I " r7=%016" PRIx64 " r8=%016" PRIx64 " r9=%016" PRIx64$
ERROR: code indent should never use tabs
#44: FILE: target/ppc/excp_helper.c:75:
+^I^I " r10=%016" PRIx64 " r11=%016" PRIx64 " r12=%016" PRIx64$
ERROR: code indent should never use tabs
#47: FILE: target/ppc/excp_helper.c:78:
+^I^I ppc_dump_gpr(env, 5), ppc_dump_gpr(env, 6),$
ERROR: code indent should never use tabs
#48: FILE: target/ppc/excp_helper.c:79:
+^I^I ppc_dump_gpr(env, 7), ppc_dump_gpr(env, 8),$
ERROR: code indent should never use tabs
#49: FILE: target/ppc/excp_helper.c:80:
+^I^I ppc_dump_gpr(env, 9), ppc_dump_gpr(env, 10),$
ERROR: code indent should never use tabs
#50: FILE: target/ppc/excp_helper.c:81:
+^I^I ppc_dump_gpr(env, 11), ppc_dump_gpr(env, 12),$
ERROR: code indent should never use tabs
#51: FILE: target/ppc/excp_helper.c:82:
+^I^I env->nip);$
total: 8 errors, 0 warnings, 47 lines checked
Patch 1/18 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/18 Checking commit d8a29c16840a (spapr: Don't check capabilities removed
between CAS calls)
3/18 Checking commit f0ed65adc034 (ppc/spapr: tweak change system reset helper)
4/18 Checking commit d9869ed76eb9 (ppc/pnv: Add support for NMI interface)
5/18 Checking commit ae65da61eb6b (spapr: Simplify selection of radix/hash
during CAS)
6/18 Checking commit 660bb297151b (spapr/cas: Separate CAS handling from
rebuilding the FDT)
7/18 Checking commit 0265febed9dd (spapr: Drop CAS reboot flag)
8/18 Checking commit 0f1ea37bf8e0 (target/ppc: Enforce that the root page
directory size must be at least 5)
9/18 Checking commit b92117299180 (target/ppc: Introduce a relocation bool in
ppc_radix64_handle_mmu_fault())
10/18 Checking commit b807de27aab6 (target/ppc: Assert if HV mode is set when
running under a pseries machine)
11/18 Checking commit 12ffc7216fdb (spapr: Don't allow unplug of NVLink2
devices)
12/18 Checking commit ba64285771b1 (target/ppc: Introduce ppc_radix64_xlate()
for Radix tree translation)
13/18 Checking commit a6648da481a2 (target/ppc: Extend ppc_radix64_check_prot()
with a 'partition_scoped' bool)
14/18 Checking commit 841c68de3923 (target/ppc: Rework ppc_radix64_walk_tree()
for partition-scoped translation)
15/18 Checking commit 3b966ed8381d (target/ppc: Add support for Radix
partition-scoped translation)
16/18 Checking commit c7118842415d (spapr_nvdimm.c: make 'label-size' mandatory)
17/18 Checking commit 8a40c9943f6e (spapr_nvdimm: Tweak error messages)
18/18 Checking commit 2d7cf2f3ab64 (target-ppc: fix rlwimi, rlwinm, rlwnm for
Clang-9)
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden
- [PULL 09/18] target/ppc: Introduce a relocation bool in ppc_radix64_handle_mmu_fault(), (continued)
- [PULL 09/18] target/ppc: Introduce a relocation bool in ppc_radix64_handle_mmu_fault(), David Gibson, 2020/05/07
- [PULL 15/18] target/ppc: Add support for Radix partition-scoped translation, David Gibson, 2020/05/07
- [PULL 08/18] target/ppc: Enforce that the root page directory size must be at least 5, David Gibson, 2020/05/07
- [PULL 16/18] spapr_nvdimm.c: make 'label-size' mandatory, David Gibson, 2020/05/07
- [PULL 17/18] spapr_nvdimm: Tweak error messages, David Gibson, 2020/05/07
- [PULL 13/18] target/ppc: Extend ppc_radix64_check_prot() with a 'partition_scoped' bool, David Gibson, 2020/05/07
- [PULL 10/18] target/ppc: Assert if HV mode is set when running under a pseries machine, David Gibson, 2020/05/07
- [PULL 12/18] target/ppc: Introduce ppc_radix64_xlate() for Radix tree translation, David Gibson, 2020/05/07
- [PULL 14/18] target/ppc: Rework ppc_radix64_walk_tree() for partition-scoped translation, David Gibson, 2020/05/07
- [PULL 18/18] target-ppc: fix rlwimi, rlwinm, rlwnm for Clang-9, David Gibson, 2020/05/07
- Re: [PULL 00/18] ppc-for-5.1 queue 20200507,
no-reply <=
- Re: [PULL 00/18] ppc-for-5.1 queue 20200507, Peter Maydell, 2020/05/07