From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [omap-audio:peter/linux-next-wip 37/89] ERROR: switch and case should be at the same indent
Date: Thu, 21 May 2020 08:41:07 +0800 [thread overview]
Message-ID: <20200521004106.GC15716@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 2230 bytes --]
tree: https://github.com/omap-audio/linux-audio peter/linux-next-wip
head: 1352930850986516f555d53639f2b6a5c4f308cf
commit: dcca0275f838f287fbbfef48909d0beec92e998e [37/89] ASoC: Initial driver for TAS2555 Smart Amplifier
reproduce: scripts/checkpatch.pl
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
# many are suggestions rather than must-fix
WARNING: DT bindings should be in DT schema format. See: Documentation/devicetree/writing-schema.rst
WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.txt
WARNING: please write a paragraph that describes the config symbol fully
#145: sound/soc/codecs/Kconfig:1262:
+config SND_SOC_TAS2555
ERROR: that open brace { should be on the previous line
#397: sound/soc/codecs/tas2555.c:219:
+static const struct snd_soc_dapm_widget tas2555_dapm_widgets[] =
+{
ERROR: switch and case should be at the same indent
#545: sound/soc/codecs/tas2555.c:367:
+ switch (dai->id) {
+ case 0: /* ASI1 */
[...]
+ case 1: /* ASI2 */
[...]
+ case 2: /* ASI3/ASIM */
[...]
+ default:
ERROR: switch and case should be at the same indent
#594: sound/soc/codecs/tas2555.c:416:
+ switch (dai->id) {
+ case 0: /* ASI1 */
[...]
+ case 1: /* ASI2 */
[...]
+ case 2: /* ASI3/ASIM */
[...]
+ default:
ERROR: switch and case should be at the same indent
#709: sound/soc/codecs/tas2555.c:531:
+ switch (dai->id) {
+ case 0: /* ASI1 */
[...]
+ case 1: /* ASI2 */
[...]
+ case 2: /* ASI3/ASIM */
[...]
+ default:
ERROR: space required before the open parenthesis '('
#777: sound/soc/codecs/tas2555.c:599:
+ } while(count < fw->size);
WARNING: please, no space before tabs
#1000: sound/soc/codecs/tas2555.c:822:
+// ^I^Iif (i == 0)$
WARNING: please, no space before tabs
#1001: sound/soc/codecs/tas2555.c:823:
+// ^I^I^Itas2555->supplies[i].optional = true;$
WARNING: Missing or malformed SPDX-License-Identifier tag in line 1
#1060: sound/soc/codecs/tas2555.h:1:
+/*
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
reply other threads:[~2020-05-21 0:41 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20200521004106.GC15716@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/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.