From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [morimoto-linux:fw-cleanup-2020-05-21-v1 55/189] sound/soc/meson/aiu-encoder-i2s.c:75:15: warning: variable 'val' set but not used
Date: Wed, 27 May 2020 06:43:22 +0800 [thread overview]
Message-ID: <202005270619.F9dfAFMN%lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 8284 bytes --]
tree: https://github.com/morimoto/linux fw-cleanup-2020-05-21-v1
head: 359a2b114d515701ce6fb31f140ef197655e6a55
commit: 4017508b0b90ea4b2fbee2ee7268ff8667509485 [55/189] ASoC: soc-component: merge snd_soc_component_read() and snd_soc_component_read32()
config: m68k-allmodconfig (attached as .config)
compiler: m68k-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
git checkout 4017508b0b90ea4b2fbee2ee7268ff8667509485
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>, old ones prefixed by <<):
In file included from arch/m68k/include/asm/io_mm.h:25,
from arch/m68k/include/asm/io.h:8,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from include/asm-generic/hardirq.h:13,
from ./arch/m68k/include/generated/asm/hardirq.h:1,
from include/linux/hardirq.h:9,
from include/linux/interrupt.h:11,
from include/sound/soc.h:18,
from sound/soc/meson/aiu-encoder-i2s.c:9:
arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb':
arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used [-Wunused-but-set-variable]
83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); | ^~~
arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro 'rom_out_8'
430 | rom_out_8(port, *buf++);
| ^~~~~~~~~
arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw':
arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [-Wunused-but-set-variable]
86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); | ^~~
arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro 'rom_out_be16'
448 | rom_out_be16(port, *buf++);
| ^~~~~~~~~~~~
arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw':
arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [-Wunused-but-set-variable]
90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); | ^~~
arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro 'rom_out_le16'
466 | rom_out_le16(port, *buf++);
| ^~~~~~~~~~~~
sound/soc/meson/aiu-encoder-i2s.c: In function 'aiu_encoder_i2s_setup_desc':
>> sound/soc/meson/aiu-encoder-i2s.c:75:15: warning: variable 'val' set but not used [-Wunused-but-set-variable]
75 | unsigned int val;
| ^~~
--
In file included from arch/m68k/include/asm/io_mm.h:25,
from arch/m68k/include/asm/io.h:8,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from include/asm-generic/hardirq.h:13,
from ./arch/m68k/include/generated/asm/hardirq.h:1,
from include/linux/hardirq.h:9,
from include/linux/interrupt.h:11,
from include/sound/soc.h:18,
from sound/soc/meson/aiu-fifo-i2s.c:9:
arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb':
arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used [-Wunused-but-set-variable]
83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); | ^~~
arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro 'rom_out_8'
430 | rom_out_8(port, *buf++);
| ^~~~~~~~~
arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw':
arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [-Wunused-but-set-variable]
86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); | ^~~
arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro 'rom_out_be16'
448 | rom_out_be16(port, *buf++);
| ^~~~~~~~~~~~
arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw':
arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [-Wunused-but-set-variable]
90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); | ^~~
arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro 'rom_out_le16'
466 | rom_out_le16(port, *buf++);
| ^~~~~~~~~~~~
sound/soc/meson/aiu-fifo-i2s.c: In function 'aiu_fifo_i2s_trigger':
>> sound/soc/meson/aiu-fifo-i2s.c:49:15: warning: variable 'val' set but not used [-Wunused-but-set-variable]
49 | unsigned int val;
| ^~~
vim +/val +75 sound/soc/meson/aiu-encoder-i2s.c
6ae9ca9ce986bff Jerome Brunet 2020-02-13 69
6ae9ca9ce986bff Jerome Brunet 2020-02-13 70 static int aiu_encoder_i2s_setup_desc(struct snd_soc_component *component,
6ae9ca9ce986bff Jerome Brunet 2020-02-13 71 struct snd_pcm_hw_params *params)
6ae9ca9ce986bff Jerome Brunet 2020-02-13 72 {
6ae9ca9ce986bff Jerome Brunet 2020-02-13 73 /* Always operate in split (classic interleaved) mode */
6ae9ca9ce986bff Jerome Brunet 2020-02-13 74 unsigned int desc = AIU_I2S_SOURCE_DESC_MODE_SPLIT;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 @75 unsigned int val;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 76
6ae9ca9ce986bff Jerome Brunet 2020-02-13 77 /* Reset required to update the pipeline */
6ae9ca9ce986bff Jerome Brunet 2020-02-13 78 snd_soc_component_write(component, AIU_RST_SOFT, AIU_RST_SOFT_I2S_FAST);
4017508b0b90ea4 Kuninori Morimoto 2020-05-11 79 val = snd_soc_component_read(component, AIU_I2S_SYNC);
6ae9ca9ce986bff Jerome Brunet 2020-02-13 80
6ae9ca9ce986bff Jerome Brunet 2020-02-13 81 switch (params_physical_width(params)) {
6ae9ca9ce986bff Jerome Brunet 2020-02-13 82 case 16: /* Nothing to do */
6ae9ca9ce986bff Jerome Brunet 2020-02-13 83 break;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 84
6ae9ca9ce986bff Jerome Brunet 2020-02-13 85 case 32:
6ae9ca9ce986bff Jerome Brunet 2020-02-13 86 desc |= (AIU_I2S_SOURCE_DESC_MODE_24BIT |
6ae9ca9ce986bff Jerome Brunet 2020-02-13 87 AIU_I2S_SOURCE_DESC_MODE_32BIT);
6ae9ca9ce986bff Jerome Brunet 2020-02-13 88 break;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 89
6ae9ca9ce986bff Jerome Brunet 2020-02-13 90 default:
6ae9ca9ce986bff Jerome Brunet 2020-02-13 91 return -EINVAL;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 92 }
6ae9ca9ce986bff Jerome Brunet 2020-02-13 93
6ae9ca9ce986bff Jerome Brunet 2020-02-13 94 switch (params_channels(params)) {
6ae9ca9ce986bff Jerome Brunet 2020-02-13 95 case 2: /* Nothing to do */
6ae9ca9ce986bff Jerome Brunet 2020-02-13 96 break;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 97 case 8:
6ae9ca9ce986bff Jerome Brunet 2020-02-13 98 desc |= AIU_I2S_SOURCE_DESC_MODE_8CH;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 99 break;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 100 default:
6ae9ca9ce986bff Jerome Brunet 2020-02-13 101 return -EINVAL;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 102 }
6ae9ca9ce986bff Jerome Brunet 2020-02-13 103
6ae9ca9ce986bff Jerome Brunet 2020-02-13 104 snd_soc_component_update_bits(component, AIU_I2S_SOURCE_DESC,
6ae9ca9ce986bff Jerome Brunet 2020-02-13 105 AIU_I2S_SOURCE_DESC_MODE_8CH |
6ae9ca9ce986bff Jerome Brunet 2020-02-13 106 AIU_I2S_SOURCE_DESC_MODE_24BIT |
6ae9ca9ce986bff Jerome Brunet 2020-02-13 107 AIU_I2S_SOURCE_DESC_MODE_32BIT |
6ae9ca9ce986bff Jerome Brunet 2020-02-13 108 AIU_I2S_SOURCE_DESC_MODE_SPLIT,
6ae9ca9ce986bff Jerome Brunet 2020-02-13 109 desc);
6ae9ca9ce986bff Jerome Brunet 2020-02-13 110
6ae9ca9ce986bff Jerome Brunet 2020-02-13 111 return 0;
6ae9ca9ce986bff Jerome Brunet 2020-02-13 112 }
6ae9ca9ce986bff Jerome Brunet 2020-02-13 113
:::::: The code at line 75 was first introduced by commit
:::::: 6ae9ca9ce986bffe71fd0fbf9595de8500891b52 ASoC: meson: aiu: add i2s and spdif support
:::::: TO: Jerome Brunet <jbrunet@baylibre.com>
:::::: CC: Mark Brown <broonie@kernel.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 53987 bytes --]
reply other threads:[~2020-05-26 22:43 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202005270619.F9dfAFMN%lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/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.