From: kernel test robot <lkp@intel.com>
To: Sameer Pujar <spujar@nvidia.com>,
broonie@kernel.org, lgirdwood@gmail.com, robh+dt@kernel.org,
kuninori.morimoto.gx@renesas.com,
pierre-louis.bossart@linux.intel.com, perex@perex.cz,
tiwai@suse.com, thierry.reding@gmail.com, jonathanh@nvidia.com
Cc: alsa-devel@alsa-project.org, kbuild-all@lists.01.org
Subject: Re: [PATCH v3 08/13] ASoC: audio-graph: Expose helpers from audio graph
Date: Fri, 2 Oct 2020 11:06:02 +0800 [thread overview]
Message-ID: <202010021011.kh8VcGVY-lkp@intel.com> (raw)
In-Reply-To: <1601573587-15288-9-git-send-email-spujar@nvidia.com>
[-- Attachment #1: Type: text/plain, Size: 7604 bytes --]
Hi Sameer,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on tegra/for-next]
[also build test WARNING on v5.9-rc7 next-20201001]
[cannot apply to asoc/for-next robh/for-next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Sameer-Pujar/Audio-graph-card-updates-and-usage-with-Tegra210-audio/20201002-013648
base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git for-next
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/b7b97805bc967aae0ce3009c1bbf17b0f232e18b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Sameer-Pujar/Audio-graph-card-updates-and-usage-with-Tegra210-audio/20201002-013648
git checkout b7b97805bc967aae0ce3009c1bbf17b0f232e18b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
sound/soc/generic/audio-graph-card.c: In function 'soc_component_is_pcm':
sound/soc/generic/audio-graph-card.c:116:28: error: implicit declaration of function 'snd_soc_find_dai_with_mutex' [-Werror=implicit-function-declaration]
116 | struct snd_soc_dai *dai = snd_soc_find_dai_with_mutex(dlc);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
sound/soc/generic/audio-graph-card.c:116:28: warning: initialization of 'struct snd_soc_dai *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
sound/soc/generic/audio-graph-card.c: At top level:
>> sound/soc/generic/audio-graph-card.c:532:5: warning: no previous prototype for 'graph_parse_of' [-Wmissing-prototypes]
532 | int graph_parse_of(struct asoc_simple_priv *priv)
| ^~~~~~~~~~~~~~
>> sound/soc/generic/audio-graph-card.c:608:6: warning: no previous prototype for 'graph_get_dais_count' [-Wmissing-prototypes]
608 | void graph_get_dais_count(struct asoc_simple_priv *priv,
| ^~~~~~~~~~~~~~~~~~~~
>> sound/soc/generic/audio-graph-card.c:667:5: warning: no previous prototype for 'graph_card_probe' [-Wmissing-prototypes]
667 | int graph_card_probe(struct snd_soc_card *card)
| ^~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/graph_parse_of +532 sound/soc/generic/audio-graph-card.c
531
> 532 int graph_parse_of(struct asoc_simple_priv *priv)
533 {
534 struct snd_soc_card *card = simple_priv_to_card(priv);
535 struct link_info li;
536 int ret;
537
538 ret = asoc_simple_parse_widgets(card, NULL);
539 if (ret < 0)
540 return ret;
541
542 ret = asoc_simple_parse_routing(card, NULL);
543 if (ret < 0)
544 return ret;
545
546 memset(&li, 0, sizeof(li));
547 for (li.cpu = 1; li.cpu >= 0; li.cpu--) {
548 /*
549 * Detect all CPU first, and Detect all Codec 2nd.
550 *
551 * In Normal sound case, all DAIs are detected
552 * as "CPU-Codec".
553 *
554 * In DPCM sound case,
555 * all CPUs are detected as "CPU-dummy", and
556 * all Codecs are detected as "dummy-Codec".
557 * To avoid random sub-device numbering,
558 * detect "dummy-Codec" in last;
559 */
560 ret = graph_for_each_link(priv, &li,
561 graph_dai_link_of,
562 graph_dai_link_of_dpcm);
563 if (ret < 0)
564 return ret;
565 }
566
567 return asoc_simple_parse_card_name(card, NULL);
568 }
569 EXPORT_SYMBOL_GPL(graph_parse_of);
570
571 static int graph_count_noml(struct asoc_simple_priv *priv,
572 struct device_node *cpu_ep,
573 struct device_node *codec_ep,
574 struct link_info *li)
575 {
576 struct device *dev = simple_priv_to_dev(priv);
577
578 li->link += 1; /* 1xCPU-Codec */
579 li->dais += 2; /* 1xCPU + 1xCodec */
580
581 dev_dbg(dev, "Count As Normal\n");
582
583 return 0;
584 }
585
586 static int graph_count_dpcm(struct asoc_simple_priv *priv,
587 struct device_node *cpu_ep,
588 struct device_node *codec_ep,
589 struct link_info *li,
590 int dup_codec)
591 {
592 struct device *dev = simple_priv_to_dev(priv);
593
594 li->link++; /* 1xCPU-dummy */
595 li->dais++; /* 1xCPU */
596
597 if (!dup_codec && codec_ep) {
598 li->link++; /* 1xdummy-Codec */
599 li->conf++; /* 1xdummy-Codec */
600 li->dais++; /* 1xCodec */
601 }
602
603 dev_dbg(dev, "Count As DPCM\n");
604
605 return 0;
606 }
607
> 608 void graph_get_dais_count(struct asoc_simple_priv *priv,
609 struct link_info *li)
610 {
611 struct device *dev = simple_priv_to_dev(priv);
612
613 /*
614 * link_num : number of links.
615 * CPU-Codec / CPU-dummy / dummy-Codec
616 * dais_num : number of DAIs
617 * ccnf_num : number of codec_conf
618 * same number for "dummy-Codec"
619 *
620 * ex1)
621 * CPU0 --- Codec0 link : 5
622 * CPU1 --- Codec1 dais : 7
623 * CPU2 -/ ccnf : 1
624 * CPU3 --- Codec2
625 *
626 * => 5 links = 2xCPU-Codec + 2xCPU-dummy + 1xdummy-Codec
627 * => 7 DAIs = 4xCPU + 3xCodec
628 * => 1 ccnf = 1xdummy-Codec
629 *
630 * ex2)
631 * CPU0 --- Codec0 link : 5
632 * CPU1 --- Codec1 dais : 6
633 * CPU2 -/ ccnf : 1
634 * CPU3 -/
635 *
636 * => 5 links = 1xCPU-Codec + 3xCPU-dummy + 1xdummy-Codec
637 * => 6 DAIs = 4xCPU + 2xCodec
638 * => 1 ccnf = 1xdummy-Codec
639 *
640 * ex3)
641 * CPU0 --- Codec0 link : 6
642 * CPU1 -/ dais : 6
643 * CPU2 --- Codec1 ccnf : 2
644 * CPU3 -/
645 *
646 * => 6 links = 0xCPU-Codec + 4xCPU-dummy + 2xdummy-Codec
647 * => 6 DAIs = 4xCPU + 2xCodec
648 * => 2 ccnf = 2xdummy-Codec
649 *
650 * ex4)
651 * CPU0 --- Codec0 (convert-rate) link : 3
652 * CPU1 --- Codec1 dais : 4
653 * ccnf : 1
654 *
655 * => 3 links = 1xCPU-Codec + 1xCPU-dummy + 1xdummy-Codec
656 * => 4 DAIs = 2xCPU + 2xCodec
657 * => 1 ccnf = 1xdummy-Codec
658 */
659 graph_for_each_link(priv, li,
660 graph_count_noml,
661 graph_count_dpcm);
662 dev_dbg(dev, "link %d, dais %d, ccnf %d\n",
663 li->link, li->dais, li->conf);
664 }
665 EXPORT_SYMBOL_GPL(graph_get_dais_count);
666
> 667 int graph_card_probe(struct snd_soc_card *card)
668 {
669 struct asoc_simple_priv *priv = snd_soc_card_get_drvdata(card);
670 int ret;
671
672 ret = asoc_simple_init_hp(card, &priv->hp_jack, NULL);
673 if (ret < 0)
674 return ret;
675
676 ret = asoc_simple_init_mic(card, &priv->mic_jack, NULL);
677 if (ret < 0)
678 return ret;
679
680 return 0;
681 }
682 EXPORT_SYMBOL_GPL(graph_card_probe);
683
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65068 bytes --]
next prev parent reply other threads:[~2020-10-02 3:07 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-01 17:32 [PATCH v3 00/13] Audio graph card updates and usage with Tegra210 audio Sameer Pujar
2020-10-01 17:32 ` [PATCH v3 01/13] ASoC: soc-core: Fix component name_prefix parsing Sameer Pujar
2020-10-01 18:53 ` Michał Mirosław
2020-10-02 8:50 ` Sameer Pujar
2020-10-01 17:32 ` [PATCH v3 02/13] ASoC: soc-pcm: Get all BEs along DAPM path Sameer Pujar
2020-10-01 17:32 ` [PATCH v3 03/13] ASoC: audio-graph: Use of_node and DAI for DPCM DAI link names Sameer Pujar
2020-10-01 17:32 ` [PATCH v3 04/13] ASoC: audio-graph: Identify 'no_pcm' DAI links for DPCM Sameer Pujar
2020-10-01 17:32 ` [PATCH v3 05/13] ASoC: audio-graph: Support empty Codec endpoint Sameer Pujar
2020-10-01 17:33 ` [PATCH v3 06/13] ASoC: simple-card-utils: Expose new members for asoc_simple_priv Sameer Pujar
2020-10-02 1:45 ` Kuninori Morimoto
2020-10-02 8:59 ` Sameer Pujar
2020-10-01 17:33 ` [PATCH v3 07/13] ASoC: audio-graph: Update driver as per new exposed members Sameer Pujar
2020-10-02 1:46 ` Kuninori Morimoto
2020-10-02 9:00 ` Sameer Pujar
2020-10-01 17:33 ` [PATCH v3 08/13] ASoC: audio-graph: Expose helpers from audio graph Sameer Pujar
2020-10-02 3:06 ` kernel test robot [this message]
2020-10-02 3:07 ` kernel test robot
2020-10-01 17:33 ` [PATCH v3 09/13] ASoC: dt-bindings: tegra: Add schema for audio graph card Sameer Pujar
2020-10-06 20:34 ` Rob Herring
2020-10-07 7:13 ` Sameer Pujar
2020-10-16 5:14 ` Sameer Pujar
2020-10-16 16:42 ` Mark Brown
2020-10-01 17:33 ` [PATCH v3 10/13] ASoC: tegra: Add audio graph based card driver Sameer Pujar
2020-10-01 19:07 ` Michał Mirosław
2020-10-01 19:44 ` Jon Hunter
2020-10-01 20:57 ` Dmitry Osipenko
2020-10-01 21:02 ` Dmitry Osipenko
2020-10-02 8:51 ` Sameer Pujar
2020-10-02 5:34 ` Kuninori Morimoto
2020-10-02 9:08 ` Sameer Pujar
2020-10-04 23:20 ` Kuninori Morimoto
2020-10-01 17:33 ` [PATCH v3 11/13] arm64: defconfig: Enable Tegra audio graph " Sameer Pujar
2020-10-01 17:33 ` [PATCH v3 12/13] arm64: tegra: Audio graph header for Tegra210 Sameer Pujar
2020-10-01 17:33 ` [PATCH v3 13/13] arm64: tegra: Audio graph sound card for Jetson Nano and TX1 Sameer Pujar
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=202010021011.kh8VcGVY-lkp@intel.com \
--to=lkp@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=jonathanh@nvidia.com \
--cc=kbuild-all@lists.01.org \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=lgirdwood@gmail.com \
--cc=perex@perex.cz \
--cc=pierre-louis.bossart@linux.intel.com \
--cc=robh+dt@kernel.org \
--cc=spujar@nvidia.com \
--cc=thierry.reding@gmail.com \
--cc=tiwai@suse.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).