All of lore.kernel.org
 help / color / mirror / Atom feed
From: no-reply@patchew.org
To: huth@tuxfamily.org
Cc: S.E.Harris@kent.ac.uk, rth@twiddle.net, mrolnik@gmail.com,
	qemu-devel@nongnu.org, f4bug@amsat.org
Subject: Re: [PATCH rc6 00/30] target/avr merger
Date: Sun, 5 Jul 2020 07:29:06 -0700 (PDT)	[thread overview]
Message-ID: <159395934458.22113.4571211972557020419@d1fd068a5071> (raw)
In-Reply-To: <20200705140315.260514-1-huth@tuxfamily.org>

Patchew URL: https://patchew.org/QEMU/20200705140315.260514-1-huth@tuxfamily.org/



Hi,

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

Subject: [PATCH rc6 00/30] target/avr merger
Type: series
Message-id: 20200705140315.260514-1-huth@tuxfamily.org

=== 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/20200703184952.4918-1-f4bug@amsat.org -> patchew/20200703184952.4918-1-f4bug@amsat.org
 - [tag update]      patchew/20200703200459.23294-1-f4bug@amsat.org -> patchew/20200703200459.23294-1-f4bug@amsat.org
 - [tag update]      patchew/20200704153908.12118-1-philmd@redhat.com -> patchew/20200704153908.12118-1-philmd@redhat.com
 * [new tag]         patchew/20200705140315.260514-1-huth@tuxfamily.org -> patchew/20200705140315.260514-1-huth@tuxfamily.org
Switched to a new branch 'test'
faad775 target/avr: Add section into QEMU documentation
8857615 tests/acceptance: Test the Arduino MEGA2560 board
b564682 tests/boot-serial: Test some Arduino boards (AVR based)
d5e4699 tests/machine-none: Add AVR support
bba1e34 target/avr: Update build system
29dd29c hw/avr: Add limited support for some Arduino boards
3d1640f hw/avr: Add some ATmega microcontrollers
43b47a1 hw/avr: Add support for loading ELF/raw binaries
1ae4d9e target/avr: Register AVR support with the rest of QEMU
a523aa8 hw/misc: avr: Add limited support for power reduction device
d673e4a hw/timer: avr: Add limited support for 16-bit timer peripheral
3cb5e31 hw/char: avr: Add limited support for USART peripheral
81ef8ef target/avr: Add support for disassembling via option '-d in_asm'
e2c7a47 target/avr: Initialize TCG register variables
31793f5 target/avr: Add instruction translation - CPU main translation function
03b0b1e target/avr: Add instruction translation - MCU Control Instructions
dd26299 target/avr: Add instruction translation - Bit and Bit-test Instructions
cf3dbfd target/avr: Add instruction translation - Data Transfer Instructions
22d4651 target/avr: Add instruction translation - Branch Instructions
120a956 target/avr: Add instruction translation - Arithmetic and Logic Instructions
28a91ea target/avr: Add instruction translation - Register definitions
b61b514 target/avr: Add instruction helpers
74089c9 target/avr: Add defintions of AVR core types
9c7e675 target/avr: Introduce enumeration AVRFeature
1fd82d4 target/avr: CPU class: Add GDB support
3c788c7 target/avr: CPU class: Add migration support
cb72169 target/avr: CPU class: Add memory menagement support
0758ca1 target/avr: CPU class: Add interrupt handling support
bf938b7 target/avr: Introduce basic CPU class object
b5e3116 target/avr: Add basic parameters of the new platform

=== OUTPUT BEGIN ===
1/30 Checking commit b5e3116a3591 (target/avr: Add basic parameters of the new platform)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#42: 
new file mode 100644

total: 0 errors, 1 warnings, 115 lines checked

Patch 1/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/30 Checking commit bf938b75dc12 (target/avr: Introduce basic CPU class object)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#25: 
new file mode 100644

total: 0 errors, 1 warnings, 412 lines checked

Patch 2/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/30 Checking commit 0758ca106cc4 (target/avr: CPU class: Add interrupt handling support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#37: 
new file mode 100644

total: 0 errors, 1 warnings, 97 lines checked

Patch 3/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/30 Checking commit cb72169d4881 (target/avr: CPU class: Add memory menagement support)
5/30 Checking commit 3c788c7f7b46 (target/avr: CPU class: Add migration support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#48: 
new file mode 100644

total: 0 errors, 1 warnings, 136 lines checked

Patch 5/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
6/30 Checking commit 1fd82d47f5a7 (target/avr: CPU class: Add GDB support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#37: 
new file mode 100644

total: 0 errors, 1 warnings, 156 lines checked

Patch 6/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
7/30 Checking commit 9c7e67571337 (target/avr: Introduce enumeration AVRFeature)
8/30 Checking commit 74089c904d15 (target/avr: Add defintions of AVR core types)
9/30 Checking commit b61b514c3997 (target/avr: Add instruction helpers)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#235: 
new file mode 100644

total: 0 errors, 1 warnings, 235 lines checked

Patch 9/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
10/30 Checking commit 28a91eae7169 (target/avr: Add instruction translation - Register definitions)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#19: 
new file mode 100644

total: 0 errors, 1 warnings, 143 lines checked

Patch 10/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
11/30 Checking commit 120a95627611 (target/avr: Add instruction translation - Arithmetic and Logic Instructions)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#27: 
new file mode 100644

total: 0 errors, 1 warnings, 908 lines checked

Patch 11/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
12/30 Checking commit 22d465149871 (target/avr: Add instruction translation - Branch Instructions)
13/30 Checking commit cf3dbfd9661c (target/avr: Add instruction translation - Data Transfer Instructions)
14/30 Checking commit dd262992399f (target/avr: Add instruction translation - Bit and Bit-test Instructions)
15/30 Checking commit 03b0b1ec78c3 (target/avr: Add instruction translation - MCU Control Instructions)
16/30 Checking commit 31793f55b06e (target/avr: Add instruction translation - CPU main translation function)
17/30 Checking commit e2c7a4769a94 (target/avr: Initialize TCG register variables)
18/30 Checking commit 81ef8efbea3c (target/avr: Add support for disassembling via option '-d in_asm')
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#71: 
new file mode 100644

total: 0 errors, 1 warnings, 279 lines checked

Patch 18/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
19/30 Checking commit 3cb5e313494e (hw/char: avr: Add limited support for USART peripheral)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#69: 
new file mode 100644

total: 0 errors, 1 warnings, 442 lines checked

Patch 19/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
20/30 Checking commit d673e4a56319 (hw/timer: avr: Add limited support for 16-bit timer peripheral)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#60: 
new file mode 100644

total: 0 errors, 1 warnings, 737 lines checked

Patch 20/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
21/30 Checking commit a523aa88b34c (hw/misc: avr: Add limited support for power reduction device)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#58: 
new file mode 100644

total: 0 errors, 1 warnings, 189 lines checked

Patch 21/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
22/30 Checking commit 1ae4d9e3bbc3 (target/avr: Register AVR support with the rest of QEMU)
23/30 Checking commit 43b47a14c9e9 (hw/avr: Add support for loading ELF/raw binaries)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#32: 
new file mode 100644

total: 0 errors, 1 warnings, 173 lines checked

Patch 23/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
24/30 Checking commit 3d1640fb8c1a (hw/avr: Add some ATmega microcontrollers)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#30: 
new file mode 100644

total: 0 errors, 1 warnings, 513 lines checked

Patch 24/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
25/30 Checking commit 29dd29ce1df9 (hw/avr: Add limited support for some Arduino boards)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#81: 
new file mode 100644

total: 0 errors, 1 warnings, 178 lines checked

Patch 25/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
26/30 Checking commit bba1e34ecbaa (target/avr: Update build system)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#57: 
new file mode 100644

total: 0 errors, 1 warnings, 65 lines checked

Patch 26/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
27/30 Checking commit d5e469951cf6 (tests/machine-none: Add AVR support)
28/30 Checking commit b564682b6339 (tests/boot-serial: Test some Arduino boards (AVR based))
29/30 Checking commit 88576153f639 (tests/acceptance: Test the Arduino MEGA2560 board)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#46: 
new file mode 100644

total: 0 errors, 1 warnings, 57 lines checked

Patch 29/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
30/30 Checking commit faad77504d50 (target/avr: Add section into QEMU documentation)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#15: 
new file mode 100644

ERROR: trailing whitespace
#39: FILE: docs/system/target-avr.rst:20:
+ - Continuous non interrupted execution: $

total: 1 errors, 1 warnings, 41 lines checked

Patch 30/30 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
http://patchew.org/logs/20200705140315.260514-1-huth@tuxfamily.org/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com

  parent reply	other threads:[~2020-07-05 14:30 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-05 14:02 [PATCH rc6 00/30] target/avr merger Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 01/30] target/avr: Add basic parameters of the new platform Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 02/30] target/avr: Introduce basic CPU class object Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 03/30] target/avr: CPU class: Add interrupt handling support Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 04/30] target/avr: CPU class: Add memory menagement support Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 05/30] target/avr: CPU class: Add migration support Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 06/30] target/avr: CPU class: Add GDB support Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 07/30] target/avr: Introduce enumeration AVRFeature Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 08/30] target/avr: Add defintions of AVR core types Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 09/30] target/avr: Add instruction helpers Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 10/30] target/avr: Add instruction translation - Register definitions Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 11/30] target/avr: Add instruction translation - Arithmetic and Logic Instructions Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 12/30] target/avr: Add instruction translation - Branch Instructions Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 13/30] target/avr: Add instruction translation - Data Transfer Instructions Thomas Huth
2020-07-05 14:02 ` [PATCH rc6 14/30] target/avr: Add instruction translation - Bit and Bit-test Instructions Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 15/30] target/avr: Add instruction translation - MCU Control Instructions Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 16/30] target/avr: Add instruction translation - CPU main translation function Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 17/30] target/avr: Initialize TCG register variables Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 18/30] target/avr: Add support for disassembling via option '-d in_asm' Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 19/30] hw/char: avr: Add limited support for USART peripheral Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 20/30] hw/timer: avr: Add limited support for 16-bit timer peripheral Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 21/30] hw/misc: avr: Add limited support for power reduction device Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 22/30] target/avr: Register AVR support with the rest of QEMU Thomas Huth
2020-07-06 14:56   ` Eric Blake
2020-07-06 16:36     ` Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 23/30] hw/avr: Add support for loading ELF/raw binaries Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 24/30] hw/avr: Add some ATmega microcontrollers Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 25/30] hw/avr: Add limited support for some Arduino boards Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 26/30] target/avr: Update build system Thomas Huth
2020-07-05 21:30   ` Philippe Mathieu-Daudé
2020-07-05 14:03 ` [PATCH rc6 27/30] tests/machine-none: Add AVR support Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 28/30] tests/boot-serial: Test some Arduino boards (AVR based) Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 29/30] tests/acceptance: Test the Arduino MEGA2560 board Thomas Huth
2020-07-05 14:03 ` [PATCH rc6 30/30] target/avr: Add section into QEMU documentation Thomas Huth
2020-07-05 14:29 ` no-reply [this message]
2020-07-05 18:30   ` [PATCH rc6 00/30] target/avr merger Thomas Huth
2020-07-05 18:42     ` Peter Maydell
2020-07-07 17:57 ` Philippe Mathieu-Daudé

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=159395934458.22113.4571211972557020419@d1fd068a5071 \
    --to=no-reply@patchew.org \
    --cc=S.E.Harris@kent.ac.uk \
    --cc=f4bug@amsat.org \
    --cc=huth@tuxfamily.org \
    --cc=mrolnik@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.