From: Ben Dooks <ben.dooks@codethink.co.uk>
To: kbuild test robot <lkp@intel.com>
Cc: Jorge Sanjuan <jorge.sanjuan@codethink.co.uk>,
linux-kernel@lists.codethink.co.uk, alsa-devel@alsa-project.org,
linux-kernel@vger.kernel.org, lgirdwood@gmail.com,
jonathanh@nvidia.com, broonie@kernel.org,
thierry.reding@gmail.com, kbuild-all@01.org,
linux-tegra@vger.kernel.org
Subject: Re: [PATCH 3/4] ASoC: tegra: Allow 32-bit and 24-bit samples
Date: Sun, 29 Jul 2018 10:21:14 +0100 [thread overview]
Message-ID: <0e990da63c37ef2bc0babc9ee96ef798@codethink.co.uk> (raw)
In-Reply-To: <201807290647.ybnC01Ii%fengguang.wu@intel.com>
On 2018-07-28 23:28, kbuild test robot wrote:
> Hi Edward,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on tegra/for-next]
> [also build test ERROR on v4.18-rc6 next-20180727]
> [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/Jorge-Sanjuan/ASoC-Tegra30-TDM-support/20180728-163720
> base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git
> for-next
> config: arm-multi_v7_defconfig (attached as .config)
> compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
> reproduce:
> wget
> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
> -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # save the attached .config to linux build tree
> GCC_VERSION=7.2.0 make.cross ARCH=arm
>
> Note: the
> linux-review/Jorge-Sanjuan/ASoC-Tegra30-TDM-support/20180728-163720
> HEAD 14bbc96df0fa027f7bc057eb2da8181baff4e22c builds fine.
> It only hurts bisectibility.
>
> All errors (new ones prefixed by >>):
>
> sound/soc/tegra/tegra30_i2s.c: In function 'tegra30_i2s_hw_params':
>>> sound/soc/tegra/tegra30_i2s.c:155:3: error: 'audio_bits' undeclared
>>> (first use in this function); did you mean 'audit_names'?
> audio_bits = TEGRA30_AUDIOCIF_BITS_24;
> ^~~~~~~~~~
> audit_names
> sound/soc/tegra/tegra30_i2s.c:155:3: note: each undeclared
> identifier is reported only once for each function it appears in
>
> vim +155 sound/soc/tegra/tegra30_i2s.c
>
> 133
> 134 static int tegra30_i2s_hw_params(struct snd_pcm_substream
> *substream,
> 135 struct snd_pcm_hw_params *params,
> 136 struct snd_soc_dai *dai)
> 137 {
> 138 struct device *dev = dai->dev;
> 139 struct tegra30_i2s *i2s = snd_soc_dai_get_drvdata(dai);
> 140 unsigned int mask, val, reg;
> 141 int ret, sample_size, srate, i2sclock, bitcnt;
> 142 struct tegra30_ahub_cif_conf cif_conf;
> 143
> 144 if (params_channels(params) != 2)
> 145 return -EINVAL;
> 146
> 147 mask = TEGRA30_I2S_CTRL_BIT_SIZE_MASK;
> 148 switch (params_format(params)) {
> 149 case SNDRV_PCM_FORMAT_S16_LE:
> 150 val = TEGRA30_I2S_CTRL_BIT_SIZE_16;
> 151 sample_size = 16;
> 152 break;
> 153 case SNDRV_PCM_FORMAT_S24_LE:
> 154 val = TEGRA30_I2S_CTRL_BIT_SIZE_24;
> > 155 audio_bits = TEGRA30_AUDIOCIF_BITS_24;
> 156 sample_size = 24;
> 157 break;
> 158 case SNDRV_PCM_FORMAT_S32_LE:
> 159 val = TEGRA30_I2S_CTRL_BIT_SIZE_32;
> 160 sample_size = 32;
> 161 break;
> 162 default:
> 163 return -EINVAL;
> 164 }
>
looks like we failed to merge in a fix from later in the internal
series we have.
jorge: can we get the channel fix from here into this patch and
resubmit?
commit dd439f5f0b748eba43da7f18cabec8850dcd18b1
Author: Edward Cragg <edward.cragg@codethink.co.uk>
Date: Thu Sep 15 17:01:49 2016 +0100
ASoC: tegra: i2s: Add support for more than 2 channels
WARNING: multiple messages have this Message-ID (diff)
From: Ben Dooks <ben.dooks@codethink.co.uk>
To: kbuild test robot <lkp@intel.com>
Cc: Jorge Sanjuan <jorge.sanjuan@codethink.co.uk>,
linux-kernel@lists.codethink.co.uk, alsa-devel@alsa-project.org,
linux-kernel@vger.kernel.org, lgirdwood@gmail.com,
jonathanh@nvidia.com, broonie@kernel.org,
thierry.reding@gmail.com, kbuild-all@01.org,
linux-tegra@vger.kernel.org
Subject: Re: [Linux-kernel] [PATCH 3/4] ASoC: tegra: Allow 32-bit and 24-bit samples
Date: Sun, 29 Jul 2018 10:21:14 +0100 [thread overview]
Message-ID: <0e990da63c37ef2bc0babc9ee96ef798@codethink.co.uk> (raw)
In-Reply-To: <201807290647.ybnC01Ii%fengguang.wu@intel.com>
On 2018-07-28 23:28, kbuild test robot wrote:
> Hi Edward,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on tegra/for-next]
> [also build test ERROR on v4.18-rc6 next-20180727]
> [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/Jorge-Sanjuan/ASoC-Tegra30-TDM-support/20180728-163720
> base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git
> for-next
> config: arm-multi_v7_defconfig (attached as .config)
> compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
> reproduce:
> wget
> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
> -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # save the attached .config to linux build tree
> GCC_VERSION=7.2.0 make.cross ARCH=arm
>
> Note: the
> linux-review/Jorge-Sanjuan/ASoC-Tegra30-TDM-support/20180728-163720
> HEAD 14bbc96df0fa027f7bc057eb2da8181baff4e22c builds fine.
> It only hurts bisectibility.
>
> All errors (new ones prefixed by >>):
>
> sound/soc/tegra/tegra30_i2s.c: In function 'tegra30_i2s_hw_params':
>>> sound/soc/tegra/tegra30_i2s.c:155:3: error: 'audio_bits' undeclared
>>> (first use in this function); did you mean 'audit_names'?
> audio_bits = TEGRA30_AUDIOCIF_BITS_24;
> ^~~~~~~~~~
> audit_names
> sound/soc/tegra/tegra30_i2s.c:155:3: note: each undeclared
> identifier is reported only once for each function it appears in
>
> vim +155 sound/soc/tegra/tegra30_i2s.c
>
> 133
> 134 static int tegra30_i2s_hw_params(struct snd_pcm_substream
> *substream,
> 135 struct snd_pcm_hw_params *params,
> 136 struct snd_soc_dai *dai)
> 137 {
> 138 struct device *dev = dai->dev;
> 139 struct tegra30_i2s *i2s = snd_soc_dai_get_drvdata(dai);
> 140 unsigned int mask, val, reg;
> 141 int ret, sample_size, srate, i2sclock, bitcnt;
> 142 struct tegra30_ahub_cif_conf cif_conf;
> 143
> 144 if (params_channels(params) != 2)
> 145 return -EINVAL;
> 146
> 147 mask = TEGRA30_I2S_CTRL_BIT_SIZE_MASK;
> 148 switch (params_format(params)) {
> 149 case SNDRV_PCM_FORMAT_S16_LE:
> 150 val = TEGRA30_I2S_CTRL_BIT_SIZE_16;
> 151 sample_size = 16;
> 152 break;
> 153 case SNDRV_PCM_FORMAT_S24_LE:
> 154 val = TEGRA30_I2S_CTRL_BIT_SIZE_24;
> > 155 audio_bits = TEGRA30_AUDIOCIF_BITS_24;
> 156 sample_size = 24;
> 157 break;
> 158 case SNDRV_PCM_FORMAT_S32_LE:
> 159 val = TEGRA30_I2S_CTRL_BIT_SIZE_32;
> 160 sample_size = 32;
> 161 break;
> 162 default:
> 163 return -EINVAL;
> 164 }
>
looks like we failed to merge in a fix from later in the internal
series we have.
jorge: can we get the channel fix from here into this patch and
resubmit?
commit dd439f5f0b748eba43da7f18cabec8850dcd18b1
Author: Edward Cragg <edward.cragg@codethink.co.uk>
Date: Thu Sep 15 17:01:49 2016 +0100
ASoC: tegra: i2s: Add support for more than 2 channels
next prev parent reply other threads:[~2018-07-29 9:21 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-27 12:59 [PATCH 0/4] ASoC: Tegra30 TDM support Jorge Sanjuan
2018-07-27 12:59 ` Jorge Sanjuan
2018-07-27 12:59 ` [PATCH 1/4] ASoC: tegra: i2s: Fix typo/broken macro Jorge Sanjuan
2018-07-27 12:59 ` Jorge Sanjuan
2018-07-30 8:58 ` Jon Hunter
2018-07-30 8:58 ` Jon Hunter
2018-07-30 11:04 ` Applied "ASoC: tegra: i2s: Fix typo/broken macro" to the asoc tree Mark Brown
2018-07-30 11:04 ` Mark Brown
2018-07-30 11:04 ` Mark Brown
2018-07-27 12:59 ` [PATCH 2/4] ASoC: tegra: Add a TDM configuration callback Jorge Sanjuan
2018-07-30 8:49 ` Mark Brown
2018-07-30 8:49 ` Mark Brown
2018-07-30 9:04 ` Ben Dooks
2018-07-30 9:04 ` [alsa-devel] " Ben Dooks
2018-07-30 9:31 ` Jon Hunter
2018-07-30 9:31 ` Jon Hunter
2018-07-30 10:18 ` Mark Brown
2018-07-30 10:18 ` Mark Brown
2018-07-30 14:04 ` Jon Hunter
2018-07-30 14:04 ` Jon Hunter
2018-07-30 14:15 ` Jon Hunter
2018-07-30 14:15 ` Jon Hunter
2018-07-30 15:07 ` Mark Brown
2018-07-30 15:07 ` Mark Brown
2018-07-30 17:39 ` [Linux-kernel] " Ben Dooks
2018-07-30 17:39 ` Ben Dooks
2018-07-27 12:59 ` [PATCH 3/4] ASoC: tegra: Allow 32-bit and 24-bit samples Jorge Sanjuan
2018-07-27 12:59 ` Jorge Sanjuan
2018-07-28 22:28 ` kbuild test robot
2018-07-28 22:28 ` kbuild test robot
2018-07-29 9:21 ` Ben Dooks [this message]
2018-07-29 9:21 ` [Linux-kernel] " Ben Dooks
2018-07-27 12:59 ` [PATCH 4/4] ASoC: tegra: i2s: Add support for more than 2 channels Jorge Sanjuan
2018-07-30 9:46 ` Jon Hunter
2018-07-30 9:46 ` Jon Hunter
2018-07-30 10:21 ` Mark Brown
2018-07-30 17:22 ` [PATCH 0/4] ASoC: Tegra30 TDM support Ben Dooks
2018-07-30 17:22 ` [Linux-kernel] " Ben Dooks
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=0e990da63c37ef2bc0babc9ee96ef798@codethink.co.uk \
--to=ben.dooks@codethink.co.uk \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=jonathanh@nvidia.com \
--cc=jorge.sanjuan@codethink.co.uk \
--cc=kbuild-all@01.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@lists.codethink.co.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=lkp@intel.com \
--cc=thierry.reding@gmail.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.