| Message ID | cover.1548714862.git.DirtY.iCE.hu@gmail.com |
|---|---|
| Headers | show |
| Series | Audio patches (was: Audio 5.1 patches) | expand |
Patchew URL: https://patchew.org/QEMU/cover.1548714862.git.DirtY.iCE.hu@gmail.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [PATCH v4 00/14] Audio patches (was: Audio 5.1 patches)
Type: series
Message-id: cover.1548714862.git.DirtY.iCE.hu@gmail.com
=== TEST SCRIPT BEGIN ===
#!/bin/bash
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'
9505d0ef85 audio: -audiodev command line option: cleanup
2f7a6267e8 wavaudio: port to -audiodev config
53f2b38246 spiceaudio: port to -audiodev config
83031a0586 sdlaudio: port to -audiodev config
a4cf2daf95 paaudio: port to -audiodev config
eca548aefa ossaudio: port to -audiodev config
96b7a11f8e noaudio: port to -audiodev config
01d330b3a7 dsoundaudio: port to -audiodev config
3738935850 coreaudio: port to -audiodev config
a2c88d6cd0 alsaaudio: port to -audiodev config
54e5804b4c audio: -audiodev command line option basic implementation
172a851936 audio: -audiodev command line option: documentation
a418745d8a audio: use qapi AudioFormat instead of audfmt_e
820a94e64b qapi: qapi for audio backends
=== OUTPUT BEGIN ===
1/14 Checking commit 820a94e64b20 (qapi: qapi for audio backends)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#51:
new file mode 100644
total: 0 errors, 1 warnings, 246 lines checked
Patch 1/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/14 Checking commit a418745d8ae2 (audio: use qapi AudioFormat instead of audfmt_e)
ERROR: space prohibited between function name and open parenthesis '('
#36: FILE: audio/alsaaudio.c:297:
+static snd_pcm_format_t aud_to_alsafmt (AudioFormat fmt, int endianness)
ERROR: space prohibited between function name and open parenthesis '('
#84: FILE: audio/alsaaudio.c:347:
+static int alsa_to_audfmt (snd_pcm_format_t alsafmt, AudioFormat *fmt,
ERROR: space prohibited between function name and open parenthesis '('
#205: FILE: audio/audio.c:260:
+static const char *audio_audfmt_to_string (AudioFormat fmt)
ERROR: space prohibited between function name and open parenthesis '('
#241: FILE: audio/audio.c:289:
+static AudioFormat audio_string_to_audfmt (const char *s, AudioFormat defval,
ERROR: space prohibited between function name and open parenthesis '('
#282: FILE: audio/audio.c:324:
+static AudioFormat audio_get_conf_fmt (const char *envname,
ERROR: space prohibited between function name and open parenthesis '('
#523: FILE: audio/ossaudio.c:151:
+static int aud_to_ossfmt (AudioFormat fmt, int endianness)
ERROR: space prohibited between function name and open parenthesis '('
#553: FILE: audio/ossaudio.c:185:
+static int oss_to_audfmt (int ossfmt, AudioFormat *fmt, int *endianness)
ERROR: space prohibited between function name and open parenthesis '('
#620: FILE: audio/paaudio.c:388:
+static pa_sample_format_t audfmt_to_pa (AudioFormat afmt, int endianness)
ERROR: space prohibited between function name and open parenthesis '('
#649: FILE: audio/paaudio.c:413:
+static AudioFormat pa_to_audfmt (pa_sample_format_t fmt, int *endianness)
ERROR: space prohibited between function name and open parenthesis '('
#687: FILE: audio/sdlaudio.c:140:
+static int aud_to_sdlfmt (AudioFormat fmt)
ERROR: trailing statements should be on next line
#954: FILE: hw/audio/hda-codec.c:102:
+ case AC_FMT_BITS_8: as->fmt = AUDIO_FORMAT_S8; break;
ERROR: trailing statements should be on next line
#955: FILE: hw/audio/hda-codec.c:103:
+ case AC_FMT_BITS_16: as->fmt = AUDIO_FORMAT_S16; break;
ERROR: trailing statements should be on next line
#956: FILE: hw/audio/hda-codec.c:104:
+ case AC_FMT_BITS_32: as->fmt = AUDIO_FORMAT_S32; break;
ERROR: space prohibited after that open square bracket '['
#970: FILE: hw/audio/hda-codec.c:137:
+ [ AUDIO_FORMAT_U8 ] = "PCM-U8",
ERROR: space prohibited before that close square bracket ']'
#970: FILE: hw/audio/hda-codec.c:137:
+ [ AUDIO_FORMAT_U8 ] = "PCM-U8",
ERROR: space prohibited after that open square bracket '['
#971: FILE: hw/audio/hda-codec.c:138:
+ [ AUDIO_FORMAT_S8 ] = "PCM-S8",
ERROR: space prohibited before that close square bracket ']'
#971: FILE: hw/audio/hda-codec.c:138:
+ [ AUDIO_FORMAT_S8 ] = "PCM-S8",
ERROR: space prohibited after that open square bracket '['
#972: FILE: hw/audio/hda-codec.c:139:
+ [ AUDIO_FORMAT_U16 ] = "PCM-U16",
ERROR: space prohibited before that close square bracket ']'
#972: FILE: hw/audio/hda-codec.c:139:
+ [ AUDIO_FORMAT_U16 ] = "PCM-U16",
ERROR: space prohibited after that open square bracket '['
#973: FILE: hw/audio/hda-codec.c:140:
+ [ AUDIO_FORMAT_S16 ] = "PCM-S16",
ERROR: space prohibited before that close square bracket ']'
#973: FILE: hw/audio/hda-codec.c:140:
+ [ AUDIO_FORMAT_S16 ] = "PCM-S16",
ERROR: space prohibited after that open square bracket '['
#974: FILE: hw/audio/hda-codec.c:141:
+ [ AUDIO_FORMAT_U32 ] = "PCM-U32",
ERROR: space prohibited before that close square bracket ']'
#974: FILE: hw/audio/hda-codec.c:141:
+ [ AUDIO_FORMAT_U32 ] = "PCM-U32",
ERROR: space prohibited after that open square bracket '['
#975: FILE: hw/audio/hda-codec.c:142:
+ [ AUDIO_FORMAT_S32 ] = "PCM-S32",
ERROR: space prohibited before that close square bracket ']'
#975: FILE: hw/audio/hda-codec.c:142:
+ [ AUDIO_FORMAT_S32 ] = "PCM-S32",
ERROR: trailing statements should be on next line
#1197: FILE: ui/vnc.c:2431:
+ case 0: vs->as.fmt = AUDIO_FORMAT_U8; break;
ERROR: trailing statements should be on next line
#1198: FILE: ui/vnc.c:2432:
+ case 1: vs->as.fmt = AUDIO_FORMAT_S8; break;
ERROR: trailing statements should be on next line
#1199: FILE: ui/vnc.c:2433:
+ case 2: vs->as.fmt = AUDIO_FORMAT_U16; break;
ERROR: trailing statements should be on next line
#1200: FILE: ui/vnc.c:2434:
+ case 3: vs->as.fmt = AUDIO_FORMAT_S16; break;
ERROR: trailing statements should be on next line
#1201: FILE: ui/vnc.c:2435:
+ case 4: vs->as.fmt = AUDIO_FORMAT_U32; break;
ERROR: trailing statements should be on next line
#1202: FILE: ui/vnc.c:2436:
+ case 5: vs->as.fmt = AUDIO_FORMAT_S32; break;
total: 31 errors, 0 warnings, 1028 lines checked
Patch 2/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/14 Checking commit 172a8519365e (audio: -audiodev command line option: documentation)
4/14 Checking commit 54e5804b4c2a (audio: -audiodev command line option basic implementation)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#937:
new file mode 100644
total: 0 errors, 1 warnings, 1283 lines checked
Patch 4/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
5/14 Checking commit a2c88d6cd00e (alsaaudio: port to -audiodev config)
6/14 Checking commit 373893585072 (coreaudio: port to -audiodev config)
7/14 Checking commit 01d330b3a7f0 (dsoundaudio: port to -audiodev config)
8/14 Checking commit 96b7a11f8ee6 (noaudio: port to -audiodev config)
9/14 Checking commit eca548aefae2 (ossaudio: port to -audiodev config)
10/14 Checking commit a4cf2daf954f (paaudio: port to -audiodev config)
11/14 Checking commit 83031a0586cb (sdlaudio: port to -audiodev config)
12/14 Checking commit 53f2b3824652 (spiceaudio: port to -audiodev config)
13/14 Checking commit 2f7a6267e87e (wavaudio: port to -audiodev config)
14/14 Checking commit 9505d0ef8530 (audio: -audiodev command line option: cleanup)
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/cover.1548714862.git.DirtY.iCE.hu@gmail.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@redhat.com