Re: [PATCH v2 00/10] target/hexagon: introduce idef-parser

[no-reply]

Patchew URL: 20210225151856.3284701-1-ale.qemu@rev.ng/">https://patchew.org/QEMU/20210225151856.3284701-1-ale.qemu@rev.ng/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Type: series
Message-id: 20210225151856.3284701-1-ale.qemu@rev.ng
Subject: [PATCH v2 00/10] target/hexagon: introduce idef-parser

=== 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
From https://github.com/patchew-project/qemu
 - [tag update]      patchew/20210225091435.644762-1-eric.auger@redhat.com -> 
patchew/20210225091435.644762-1-eric.auger@redhat.com
 * [new tag]         patchew/20210225151856.3284701-1-ale.qemu@rev.ng -> 
patchew/20210225151856.3284701-1-ale.qemu@rev.ng
Switched to a new branch 'test'
739bb05 target/hexagon: import additional tests
c412a0e target/hexagon: call idef-parser functions
41c3d34 target/hexagon: import parser for idef-parser
6fbf5f9 target/hexagon: import lexer for idef-parser
b35ce32 target/hexagon: prepare input for the idef-parser
bfa9eac target/hexagon: expose next PC in DisasContext
3a44618 target/hexagon: introduce new helper functions
df56633 target/hexagon: make helper functions non-static
e829131 target/hexagon: import README for idef-parser
44c5544 target/hexagon: update MAINTAINERS for idef-parser

=== OUTPUT BEGIN ===
1/10 Checking commit 44c554486189 (target/hexagon: update MAINTAINERS for 
idef-parser)
2/10 Checking commit e829131d737c (target/hexagon: import README for 
idef-parser)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#38: 
new file mode 100644

total: 0 errors, 1 warnings, 464 lines checked

Patch 2/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/10 Checking commit df5663383efb (target/hexagon: make helper functions 
non-static)
4/10 Checking commit 3a44618adb7c (target/hexagon: introduce new helper 
functions)
5/10 Checking commit bfa9eacded62 (target/hexagon: expose next PC in 
DisasContext)
6/10 Checking commit b35ce3214587 (target/hexagon: prepare input for the 
idef-parser)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#20: 
new file mode 100644

total: 0 errors, 1 warnings, 341 lines checked

Patch 6/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
7/10 Checking commit 6fbf5f96ef69 (target/hexagon: import lexer for idef-parser)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#29: 
new file mode 100644

total: 0 errors, 1 warnings, 987 lines checked

Patch 7/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
8/10 Checking commit 41c3d34a00e0 (target/hexagon: import parser for 
idef-parser)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#30: 
new file mode 100644

ERROR: suspicious ; after while (0)
#3320: FILE: target/hexagon/idef-parser/parser-helpers.h:99:
+    } while (0);

total: 1 errors, 1 warnings, 3583 lines checked

Patch 8/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

9/10 Checking commit c412a0e3f999 (target/hexagon: call idef-parser functions)
10/10 Checking commit 739bb05d384e (target/hexagon: import additional tests)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#100: 
new file mode 100644

total: 0 errors, 1 warnings, 1086 lines checked

Patch 10/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
20210225151856.3284701-1-ale.qemu@rev.ng/testing.checkpatch/?type=message">http://patchew.org/logs/20210225151856.3284701-1-ale.qemu@rev.ng/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com