Patchew URL:
https://patchew.org/QEMU/cover.1548714862.git.dirty.ice...@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...@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:
+