From: kbuild test robot <lkp@intel.com> Cc: Jonathan Hunter <jonathanh@nvidia.com>, linux-kernel@lists.codethink.co.uk, alsa-devel@alsa-project.org, Liam Girdwood <lgirdwood@gmail.com>, Takashi Iwai <tiwai@suse.com>, Mark Brown <broonie@kernel.org>, Thierry Reding <thierry.reding@gmail.com>, kbuild-all@01.org, linux-tegra@vger.kernel.org, Ben Dooks <ben.dooks@codethink.co.uk> Subject: Re: [PATCH 5/8] ASoC: tegra: set edge mode for TDM correctly Date: Wed, 18 Sep 2019 03:33:07 +0800 [thread overview] Message-ID: <201909180325.mpj0DWjZ%lkp@intel.com> (raw) In-Reply-To: <20190917181233.534-6-ben.dooks@codethink.co.uk> [-- Attachment #1: Type: text/plain, Size: 3832 bytes --] Hi Ben, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [cannot apply to v5.3 next-20190916] [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/Ben-Dooks/ASoC-tegra-Add-a-TDM-configuration-callback/20190918-022251 config: mips-allmodconfig (attached as .config) compiler: mips-linux-gcc (GCC) 7.4.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.4.0 make.cross ARCH=mips If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All errors (new ones prefixed by >>): sound/soc/tegra/tegra30_i2s.c: In function 'tegra30_i2s_set_fmt': >> sound/soc/tegra/tegra30_i2s.c:121:63: error: macro "regmap_update_bits" requires 4 arguments, but only 3 given i2s->is_tdm ? TEGRA30_I2S_CH_CTRL_EGDE_CTRL_NEG_EDGE : 0); ^ >> sound/soc/tegra/tegra30_i2s.c:120:2: error: 'regmap_update_bits' undeclared (first use in this function); did you mean 'regmap_update_bits_base'? regmap_update_bits(i2s->regmap, TEGRA30_I2S_CH_CTRL_EGDE_CTRL_MASK, ^~~~~~~~~~~~~~~~~~ regmap_update_bits_base sound/soc/tegra/tegra30_i2s.c:120:2: note: each undeclared identifier is reported only once for each function it appears in vim +/regmap_update_bits +121 sound/soc/tegra/tegra30_i2s.c 64 65 static int tegra30_i2s_set_fmt(struct snd_soc_dai *dai, 66 unsigned int fmt) 67 { 68 struct tegra30_i2s *i2s = snd_soc_dai_get_drvdata(dai); 69 unsigned int mask = 0, val = 0; 70 71 switch (fmt & SND_SOC_DAIFMT_INV_MASK) { 72 case SND_SOC_DAIFMT_NB_NF: 73 break; 74 default: 75 return -EINVAL; 76 } 77 78 mask |= TEGRA30_I2S_CTRL_MASTER_ENABLE; 79 switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { 80 case SND_SOC_DAIFMT_CBS_CFS: 81 val |= TEGRA30_I2S_CTRL_MASTER_ENABLE; 82 break; 83 case SND_SOC_DAIFMT_CBM_CFM: 84 break; 85 default: 86 return -EINVAL; 87 } 88 89 i2s->is_tdm = false; 90 mask |= TEGRA30_I2S_CTRL_FRAME_FORMAT_MASK | 91 TEGRA30_I2S_CTRL_LRCK_MASK; 92 switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) { 93 case SND_SOC_DAIFMT_DSP_A: 94 i2s->is_tdm = true; 95 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_FSYNC; 96 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 97 break; 98 case SND_SOC_DAIFMT_DSP_B: 99 i2s->is_tdm = true; 100 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_FSYNC; 101 val |= TEGRA30_I2S_CTRL_LRCK_R_LOW; 102 break; 103 case SND_SOC_DAIFMT_I2S: 104 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_LRCK; 105 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 106 break; 107 case SND_SOC_DAIFMT_RIGHT_J: 108 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_LRCK; 109 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 110 break; 111 case SND_SOC_DAIFMT_LEFT_J: 112 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_LRCK; 113 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 114 break; 115 default: 116 return -EINVAL; 117 } 118 119 pm_runtime_get_sync(dai->dev); > 120 regmap_update_bits(i2s->regmap, TEGRA30_I2S_CH_CTRL_EGDE_CTRL_MASK, > 121 i2s->is_tdm ? TEGRA30_I2S_CH_CTRL_EGDE_CTRL_NEG_EDGE : 0); 122 regmap_update_bits(i2s->regmap, TEGRA30_I2S_CTRL, mask, val); 123 pm_runtime_put(dai->dev); 124 125 return 0; 126 } 127 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 61579 bytes --] [-- Attachment #3: Type: text/plain, Size: 0 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com> To: Ben Dooks <ben.dooks@codethink.co.uk> Cc: Jonathan Hunter <jonathanh@nvidia.com>, linux-kernel@lists.codethink.co.uk, alsa-devel@alsa-project.org, Liam Girdwood <lgirdwood@gmail.com>, Takashi Iwai <tiwai@suse.com>, Mark Brown <broonie@kernel.org>, Thierry Reding <thierry.reding@gmail.com>, kbuild-all@01.org, linux-tegra@vger.kernel.org, Ben Dooks <ben.dooks@codethink.co.uk> Subject: Re: [alsa-devel] [PATCH 5/8] ASoC: tegra: set edge mode for TDM correctly Date: Wed, 18 Sep 2019 03:33:07 +0800 [thread overview] Message-ID: <201909180325.mpj0DWjZ%lkp@intel.com> (raw) In-Reply-To: <20190917181233.534-6-ben.dooks@codethink.co.uk> [-- Attachment #1: Type: text/plain, Size: 3832 bytes --] Hi Ben, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [cannot apply to v5.3 next-20190916] [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/Ben-Dooks/ASoC-tegra-Add-a-TDM-configuration-callback/20190918-022251 config: mips-allmodconfig (attached as .config) compiler: mips-linux-gcc (GCC) 7.4.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.4.0 make.cross ARCH=mips If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All errors (new ones prefixed by >>): sound/soc/tegra/tegra30_i2s.c: In function 'tegra30_i2s_set_fmt': >> sound/soc/tegra/tegra30_i2s.c:121:63: error: macro "regmap_update_bits" requires 4 arguments, but only 3 given i2s->is_tdm ? TEGRA30_I2S_CH_CTRL_EGDE_CTRL_NEG_EDGE : 0); ^ >> sound/soc/tegra/tegra30_i2s.c:120:2: error: 'regmap_update_bits' undeclared (first use in this function); did you mean 'regmap_update_bits_base'? regmap_update_bits(i2s->regmap, TEGRA30_I2S_CH_CTRL_EGDE_CTRL_MASK, ^~~~~~~~~~~~~~~~~~ regmap_update_bits_base sound/soc/tegra/tegra30_i2s.c:120:2: note: each undeclared identifier is reported only once for each function it appears in vim +/regmap_update_bits +121 sound/soc/tegra/tegra30_i2s.c 64 65 static int tegra30_i2s_set_fmt(struct snd_soc_dai *dai, 66 unsigned int fmt) 67 { 68 struct tegra30_i2s *i2s = snd_soc_dai_get_drvdata(dai); 69 unsigned int mask = 0, val = 0; 70 71 switch (fmt & SND_SOC_DAIFMT_INV_MASK) { 72 case SND_SOC_DAIFMT_NB_NF: 73 break; 74 default: 75 return -EINVAL; 76 } 77 78 mask |= TEGRA30_I2S_CTRL_MASTER_ENABLE; 79 switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { 80 case SND_SOC_DAIFMT_CBS_CFS: 81 val |= TEGRA30_I2S_CTRL_MASTER_ENABLE; 82 break; 83 case SND_SOC_DAIFMT_CBM_CFM: 84 break; 85 default: 86 return -EINVAL; 87 } 88 89 i2s->is_tdm = false; 90 mask |= TEGRA30_I2S_CTRL_FRAME_FORMAT_MASK | 91 TEGRA30_I2S_CTRL_LRCK_MASK; 92 switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) { 93 case SND_SOC_DAIFMT_DSP_A: 94 i2s->is_tdm = true; 95 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_FSYNC; 96 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 97 break; 98 case SND_SOC_DAIFMT_DSP_B: 99 i2s->is_tdm = true; 100 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_FSYNC; 101 val |= TEGRA30_I2S_CTRL_LRCK_R_LOW; 102 break; 103 case SND_SOC_DAIFMT_I2S: 104 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_LRCK; 105 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 106 break; 107 case SND_SOC_DAIFMT_RIGHT_J: 108 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_LRCK; 109 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 110 break; 111 case SND_SOC_DAIFMT_LEFT_J: 112 val |= TEGRA30_I2S_CTRL_FRAME_FORMAT_LRCK; 113 val |= TEGRA30_I2S_CTRL_LRCK_L_LOW; 114 break; 115 default: 116 return -EINVAL; 117 } 118 119 pm_runtime_get_sync(dai->dev); > 120 regmap_update_bits(i2s->regmap, TEGRA30_I2S_CH_CTRL_EGDE_CTRL_MASK, > 121 i2s->is_tdm ? TEGRA30_I2S_CH_CTRL_EGDE_CTRL_NEG_EDGE : 0); 122 regmap_update_bits(i2s->regmap, TEGRA30_I2S_CTRL, mask, val); 123 pm_runtime_put(dai->dev); 124 125 return 0; 126 } 127 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 61579 bytes --] [-- Attachment #3: Type: text/plain, Size: 161 bytes --] _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2019-09-17 19:33 UTC|newest] Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-09-17 18:12 RFC: TDM mode support on the tegra30 Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-17 18:12 ` [PATCH 1/8] ASoC: tegra: Add a TDM configuration callback Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-17 18:22 ` Pierre-Louis Bossart 2019-09-17 18:22 ` [alsa-devel] " Pierre-Louis Bossart 2019-09-18 8:42 ` Jon Hunter 2019-09-18 8:42 ` [alsa-devel] " Jon Hunter 2019-09-18 10:11 ` Ben Dooks 2019-09-18 10:11 ` [alsa-devel] " Ben Dooks 2019-09-18 10:25 ` Jon Hunter 2019-09-18 10:25 ` [alsa-devel] " Jon Hunter 2019-09-18 10:48 ` Mark Brown 2019-09-18 10:48 ` [alsa-devel] " Mark Brown 2019-09-18 11:44 ` Ben Dooks 2019-09-18 11:44 ` [alsa-devel] " Ben Dooks 2019-09-18 13:33 ` Pierre-Louis Bossart 2019-09-18 13:33 ` [alsa-devel] " Pierre-Louis Bossart 2019-09-18 15:02 ` Mark Brown 2019-09-18 15:02 ` [alsa-devel] " Mark Brown 2019-09-17 18:12 ` [PATCH 2/8] ASoC: tegra: Allow 24bit and 32bit samples Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-17 18:26 ` Pierre-Louis Bossart 2019-09-17 18:26 ` [alsa-devel] " Pierre-Louis Bossart 2019-09-18 7:44 ` Ben Dooks 2019-09-18 7:44 ` [alsa-devel] " Ben Dooks 2019-09-18 10:08 ` Mark Brown 2019-09-18 10:08 ` [alsa-devel] " Mark Brown 2019-09-18 11:50 ` Ben Dooks 2019-09-18 11:50 ` [alsa-devel] " Ben Dooks 2019-09-18 12:05 ` Mark Brown 2019-09-18 12:05 ` [alsa-devel] " Mark Brown 2019-09-17 18:12 ` [PATCH 3/8] ASoC: tegra: i2s: Add support for more than 2 channels Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-18 8:50 ` Jon Hunter 2019-09-18 8:50 ` [alsa-devel] " Jon Hunter 2019-09-18 10:12 ` Ben Dooks 2019-09-18 10:12 ` [alsa-devel] " Ben Dooks 2019-09-17 18:12 ` [PATCH 4/8] ASoC: tegra: disable rx_fifo after disable stream Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-17 18:12 ` [PATCH 5/8] ASoC: tegra: set edge mode for TDM correctly Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-17 19:33 ` kbuild test robot [this message] 2019-09-17 19:33 ` kbuild test robot 2019-09-18 8:54 ` Jon Hunter 2019-09-18 8:54 ` [alsa-devel] " Jon Hunter 2019-09-18 9:02 ` Jon Hunter 2019-09-18 9:02 ` [alsa-devel] " Jon Hunter 2019-09-18 10:15 ` Ben Dooks 2019-09-18 10:15 ` [alsa-devel] " Ben Dooks 2019-09-17 18:12 ` [PATCH 6/8] ASoC: tegra: set i2s_offset to 0 for tdm Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-18 9:02 ` Jon Hunter 2019-09-18 9:02 ` [alsa-devel] " Jon Hunter 2019-09-18 11:30 ` Ben Dooks 2019-09-18 11:30 ` [alsa-devel] " Ben Dooks 2019-09-17 18:12 ` [PATCH 7/8] ASoC: tegra: config fifos on hw_param changes Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-18 9:14 ` Jon Hunter 2019-09-18 9:14 ` [alsa-devel] " Jon Hunter 2019-09-18 11:41 ` Ben Dooks 2019-09-18 11:41 ` [alsa-devel] " Ben Dooks 2019-09-17 18:12 ` [PATCH 8/8] ASoC: tegra: take packing settings from the audio cif_config Ben Dooks 2019-09-17 18:12 ` [alsa-devel] " Ben Dooks 2019-09-18 9:16 ` Jon Hunter 2019-09-18 9:16 ` [alsa-devel] " Jon Hunter 2019-09-18 10:02 ` Sameer Pujar 2019-09-18 10:02 ` [alsa-devel] " 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=201909180325.mpj0DWjZ%lkp@intel.com \ --to=lkp@intel.com \ --cc=alsa-devel@alsa-project.org \ --cc=ben.dooks@codethink.co.uk \ --cc=broonie@kernel.org \ --cc=jonathanh@nvidia.com \ --cc=kbuild-all@01.org \ --cc=lgirdwood@gmail.com \ --cc=linux-kernel@lists.codethink.co.uk \ --cc=linux-tegra@vger.kernel.org \ --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: linkBe 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.