From: kbuild test robot <lkp@intel•com>
Cc: kbuild-all@01•org, broonie@kernel•org,
alsa-devel@alsa-project•org, rdunlap@infradead•org,
sfr@canb•auug.org.au, liam.r.girdwood@intel•com,
yang.jie@intel•com, linux-next@vger•kernel.org,
linux-kernel@vger•kernel.org, vinod.koul@intel•com
Subject: Re: [PATCH] ASoC: Intel: sst: only use sst-firmware when DW DMAC is available
Date: Wed, 21 Oct 2015 21:36:05 +0800 [thread overview]
Message-ID: <201510212125.qgN77JBf%fengguang.wu@intel.com> (raw)
In-Reply-To: <1445430785-27554-1-git-send-email-yang.jie@intel.com>
[-- Attachment #1: Type: text/plain, Size: 8574 bytes --]
Hi Jie,
[auto build test ERROR on asoc/for-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
url: https://github.com/0day-ci/linux/commits/Jie-Yang/ASoC-Intel-sst-only-use-sst-firmware-when-DW-DMAC-is-available/20151021-203247
config: i386-allmodconfig (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All error/warnings (new ones prefixed by >>):
sound/soc/intel/baytrail/sst-baytrail-ipc.c: In function 'sst_byt_dsp_init':
>> sound/soc/intel/baytrail/sst-baytrail-ipc.c:725:13: error: implicit declaration of function 'sst_dsp_new' [-Werror=implicit-function-declaration]
byt->dsp = sst_dsp_new(dev, &byt_dev, pdata);
^
>> sound/soc/intel/baytrail/sst-baytrail-ipc.c:725:11: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
byt->dsp = sst_dsp_new(dev, &byt_dev, pdata);
^
>> sound/soc/intel/baytrail/sst-baytrail-ipc.c:771:2: error: implicit declaration of function 'sst_dsp_free' [-Werror=implicit-function-declaration]
sst_dsp_free(byt->dsp);
^
cc1: some warnings being treated as errors
--
sound/soc/intel/haswell/sst-haswell-ipc.c: In function 'sst_hsw_dsp_init':
>> sound/soc/intel/haswell/sst-haswell-ipc.c:2148:13: error: implicit declaration of function 'sst_dsp_new' [-Werror=implicit-function-declaration]
hsw->dsp = sst_dsp_new(dev, &hsw_dev, pdata);
^
>> sound/soc/intel/haswell/sst-haswell-ipc.c:2148:11: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hsw->dsp = sst_dsp_new(dev, &hsw_dev, pdata);
^
>> sound/soc/intel/haswell/sst-haswell-ipc.c:2218:2: error: implicit declaration of function 'sst_dsp_free' [-Werror=implicit-function-declaration]
sst_dsp_free(hsw->dsp);
^
cc1: some warnings being treated as errors
vim +/sst_dsp_new +725 sound/soc/intel/baytrail/sst-baytrail-ipc.c
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 719
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 720 INIT_LIST_HEAD(&byt->stream_list);
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 721 init_waitqueue_head(&byt->boot_wait);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 722 byt_dev.thread_context = byt;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 723
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 724 /* init SST shim */
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 @725 byt->dsp = sst_dsp_new(dev, &byt_dev, pdata);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 726 if (byt->dsp == NULL) {
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 727 err = -ENODEV;
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 728 goto dsp_new_err;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 729 }
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 730
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 731 ipc->dsp = byt->dsp;
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 732
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 733 /* keep the DSP in reset state for base FW loading */
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 734 sst_dsp_reset(byt->dsp);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 735
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 736 byt_sst_fw = sst_fw_new(byt->dsp, pdata->fw, byt);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 737 if (byt_sst_fw == NULL) {
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 738 err = -ENODEV;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 739 dev_err(dev, "error: failed to load firmware\n");
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 740 goto fw_err;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 741 }
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 742
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 743 /* wait for DSP boot completion */
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 744 sst_dsp_boot(byt->dsp);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 745 err = wait_event_timeout(byt->boot_wait, byt->boot_complete,
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 746 msecs_to_jiffies(IPC_BOOT_MSECS));
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 747 if (err == 0) {
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 748 err = -EIO;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 749 dev_err(byt->dev, "ipc: error DSP boot timeout\n");
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 750 goto boot_err;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 751 }
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 752
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 753 /* show firmware information */
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 754 sst_dsp_inbox_read(byt->dsp, &init, sizeof(init));
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 755 dev_info(byt->dev, "FW version: %02x.%02x.%02x.%02x\n",
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 756 init.fw_version.major, init.fw_version.minor,
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 757 init.fw_version.build, init.fw_version.type);
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 758 dev_info(byt->dev, "Build type: %x\n", init.fw_version.type);
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 759 dev_info(byt->dev, "Build date: %s %s\n",
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 760 init.build_info.date, init.build_info.time);
4131eceb sound/soc/intel/sst-baytrail-ipc.c Jarkko Nikula 2014-06-06 761
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 762 pdata->dsp = byt;
af94aa55 sound/soc/intel/sst-baytrail-ipc.c Liam Girdwood 2014-05-08 763 byt->fw = byt_sst_fw;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 764
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 765 return 0;
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 766
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 767 boot_err:
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 768 sst_dsp_reset(byt->dsp);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 769 sst_fw_free(byt_sst_fw);
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 770 fw_err:
a4b12990 sound/soc/intel/sst-baytrail-ipc.c Mark Brown 2014-03-12 @771 sst_dsp_free(byt->dsp);
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 772 dsp_new_err:
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 773 sst_ipc_fini(ipc);
48cec59b sound/soc/intel/baytrail/sst-baytrail-ipc.c Jin Yao 2015-04-07 774 ipc_init_err:
:::::: The code at line 725 was first introduced by commit
:::::: a4b12990b68079290ab62799035afe175b4bdc23 Merge remote-tracking branches 'asoc/topic/ml26124', 'asoc/topic/of', 'asoc/topic/omap', 'asoc/topic/pxa' and 'asoc/topic/rcar' into asoc-next
:::::: TO: Mark Brown <broonie@linaro•org>
:::::: CC: Mark Brown <broonie@linaro•org>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 51660 bytes --]
prev parent reply other threads:[~2015-10-21 13:34 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-21 12:33 [PATCH] ASoC: Intel: sst: only use sst-firmware when DW DMAC is available Jie Yang
2015-10-21 13:36 ` kbuild test robot [this message]
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=201510212125.qgN77JBf%fengguang.wu@intel.com \
--to=lkp@intel$(echo .)com \
--cc=alsa-devel@alsa-project$(echo .)org \
--cc=broonie@kernel$(echo .)org \
--cc=kbuild-all@01$(echo .)org \
--cc=liam.r.girdwood@intel$(echo .)com \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-next@vger$(echo .)kernel.org \
--cc=rdunlap@infradead$(echo .)org \
--cc=sfr@canb$(echo .)auug.org.au \
--cc=vinod.koul@intel$(echo .)com \
--cc=yang.jie@intel$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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