From: kbuild test robot <lkp@intel.com>
To: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc: kbuild-all@01.org, Mark Brown <broonie@kernel.org>,
alsa-devel@alsa-project.org, Patrick Lai <plai@codeaurora.org>,
kwestfie@codeaurora.org,
Banajit Goswami <bgoswami@codeaurora.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
linux-kernel@vger.kernel.org,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Subject: Re: [PATCH v1 2/2] sound: lpass-platform: Move dma channel allocation to pcmops
Date: Mon, 13 Jun 2016 21:39:26 +0800 [thread overview]
Message-ID: <201606132118.1mfKAJSR%fengguang.wu@intel.com> (raw)
In-Reply-To: <1465824197-7735-2-git-send-email-srinivas.kandagatla@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 2229 bytes --]
Hi,
[auto build test WARNING on asoc/for-next]
[also build test WARNING on v4.7-rc3 next-20160609]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Srinivas-Kandagatla/sound-lpass-cpu-add-module-licence-and-description/20160613-212531
base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
config: sparc64-allyesconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=sparc64
Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
All warnings (new ones prefixed by >>):
sound/soc/qcom/lpass-platform.c: In function 'lpass_platform_pcmops_close':
>> sound/soc/qcom/lpass-platform.c:128:21: warning: array subscript is below array bounds [-Warray-bounds]
drvdata->substream[ch] = substream;
^
vim +128 sound/soc/qcom/lpass-platform.c
112 if (ch >= 0 && v->free_dma_channel)
113 v->free_dma_channel(drvdata, ch);
114
115 return ret;
116 }
117
118 static int lpass_platform_pcmops_close(struct snd_pcm_substream *substream)
119 {
120 struct snd_soc_pcm_runtime *soc_runtime = substream->private_data;
121 struct lpass_pcm_data *pcm_data = snd_soc_pcm_get_drvdata(soc_runtime);
122 struct lpass_data *drvdata =
123 snd_soc_platform_get_drvdata(soc_runtime->platform);
124 struct lpass_variant *v = drvdata->variant;
125 int ch = -1;
126
127 if (v->free_dma_channel) {
> 128 drvdata->substream[ch] = substream;
129
130 if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK)
131 ch = pcm_data->rdma_ch;
132 else
133 ch = pcm_data->wrdma_ch;
134
135 if (ch >= 0)
136 v->free_dma_channel(drvdata, ch);
---
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: 46475 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, Mark Brown <broonie@kernel.org>,
alsa-devel@alsa-project.org, Patrick Lai <plai@codeaurora.org>,
kwestfie@codeaurora.org,
Banajit Goswami <bgoswami@codeaurora.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
linux-kernel@vger.kernel.org,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Subject: Re: [PATCH v1 2/2] sound: lpass-platform: Move dma channel allocation to pcmops
Date: Mon, 13 Jun 2016 21:39:26 +0800 [thread overview]
Message-ID: <201606132118.1mfKAJSR%fengguang.wu@intel.com> (raw)
In-Reply-To: <1465824197-7735-2-git-send-email-srinivas.kandagatla@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 2229 bytes --]
Hi,
[auto build test WARNING on asoc/for-next]
[also build test WARNING on v4.7-rc3 next-20160609]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Srinivas-Kandagatla/sound-lpass-cpu-add-module-licence-and-description/20160613-212531
base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
config: sparc64-allyesconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=sparc64
Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
All warnings (new ones prefixed by >>):
sound/soc/qcom/lpass-platform.c: In function 'lpass_platform_pcmops_close':
>> sound/soc/qcom/lpass-platform.c:128:21: warning: array subscript is below array bounds [-Warray-bounds]
drvdata->substream[ch] = substream;
^
vim +128 sound/soc/qcom/lpass-platform.c
112 if (ch >= 0 && v->free_dma_channel)
113 v->free_dma_channel(drvdata, ch);
114
115 return ret;
116 }
117
118 static int lpass_platform_pcmops_close(struct snd_pcm_substream *substream)
119 {
120 struct snd_soc_pcm_runtime *soc_runtime = substream->private_data;
121 struct lpass_pcm_data *pcm_data = snd_soc_pcm_get_drvdata(soc_runtime);
122 struct lpass_data *drvdata =
123 snd_soc_platform_get_drvdata(soc_runtime->platform);
124 struct lpass_variant *v = drvdata->variant;
125 int ch = -1;
126
127 if (v->free_dma_channel) {
> 128 drvdata->substream[ch] = substream;
129
130 if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK)
131 ch = pcm_data->rdma_ch;
132 else
133 ch = pcm_data->wrdma_ch;
134
135 if (ch >= 0)
136 v->free_dma_channel(drvdata, ch);
---
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: 46475 bytes --]
next prev parent reply other threads:[~2016-06-13 13:40 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-13 13:23 [PATCH v1 1/2] sound: lpass-cpu: add module licence and description Srinivas Kandagatla
2016-06-13 13:23 ` [PATCH v1 2/2] sound: lpass-platform: Move dma channel allocation to pcmops Srinivas Kandagatla
2016-06-13 13:39 ` kbuild test robot [this message]
2016-06-13 13:39 ` kbuild test robot
2016-06-14 7:02 ` [PATCH v1 1/2] sound: lpass-cpu: add module licence and description Kenneth Westfield
2016-10-31 18:30 ` Applied "ASoC: lpass-cpu: add module licence and description" to the asoc tree Mark Brown
2016-10-31 18:30 ` Mark Brown
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=201606132118.1mfKAJSR%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=bgoswami@codeaurora.org \
--cc=broonie@kernel.org \
--cc=kbuild-all@01.org \
--cc=kwestfie@codeaurora.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=perex@perex.cz \
--cc=plai@codeaurora.org \
--cc=srinivas.kandagatla@linaro.org \
--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 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.