tree:   git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git 
topic/firewire
head:   9b1ee0b2cb8bffdbb3003b1d5205f3ae0592c15a
commit: 1017abed18ae7087e7f3e256c48421d09d83176e [18/49] ALSA: firewire-lib: 
Add some AV/C general commands
reproduce: make C=1 CF=-D__CHECK_ENDIAN__

>> sound/firewire/amdtp.c:199:29: sparse: cannot size expression

git remote add sound 
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git
git remote update sound
git checkout 1017abed18ae7087e7f3e256c48421d09d83176e
vim +199 sound/firewire/amdtp.c

31ef9134 Clemens Ladisch  2011-03-15  183   * changed while the stream is 
running.
31ef9134 Clemens Ladisch  2011-03-15  184   */
be4a2894 Takashi Sakamoto 2014-04-25  185  void 
amdtp_stream_set_parameters(struct amdtp_stream *s,
be4a2894 Takashi Sakamoto 2014-04-25  186                                
unsigned int rate,
be4a2894 Takashi Sakamoto 2014-04-25  187                                
unsigned int pcm_channels,
be4a2894 Takashi Sakamoto 2014-04-25  188                                
unsigned int midi_ports)
31ef9134 Clemens Ladisch  2011-03-15  189  {
77d2a8a4 Takashi Sakamoto 2014-04-25  190       unsigned int i, sfc, 
midi_channels;
31ef9134 Clemens Ladisch  2011-03-15  191  
83d8d72d Takashi Sakamoto 2014-04-25  192       midi_channels = 
DIV_ROUND_UP(midi_ports, 8);
83d8d72d Takashi Sakamoto 2014-04-25  193  
77d2a8a4 Takashi Sakamoto 2014-04-25  194       if 
(WARN_ON(amdtp_stream_running(s)) |
77d2a8a4 Takashi Sakamoto 2014-04-25  195           WARN_ON(pcm_channels > 
AMDTP_MAX_CHANNELS_FOR_PCM) |
83d8d72d Takashi Sakamoto 2014-04-25  196           WARN_ON(midi_channels > 
AMDTP_MAX_CHANNELS_FOR_MIDI))
31ef9134 Clemens Ladisch  2011-03-15  197               return;
31ef9134 Clemens Ladisch  2011-03-15  198  
1017abed Takashi Sakamoto 2014-04-25 @199       for (sfc = 0; sfc < 
sizeof(amdtp_rate_table); ++sfc)

The warning seems to be caused by a problem in sparse but anyway, this
should be "sfc < ARRAY_SIZE(amdtp_rate_table)" instead of sizeof().  The
amdtp_rate_table[] array holds integers so sizeof() is 4x larger than
we want.

1017abed Takashi Sakamoto 2014-04-25  200               if 
(amdtp_rate_table[sfc] == rate)
e84d15f6 Clemens Ladisch  2011-09-04  201                       goto sfc_found;
31ef9134 Clemens Ladisch  2011-03-15  202       WARN_ON(1);
e84d15f6 Clemens Ladisch  2011-09-04  203       return;
e84d15f6 Clemens Ladisch  2011-09-04  204  
e84d15f6 Clemens Ladisch  2011-09-04  205  sfc_found:
10550bea Takashi Sakamoto 2014-04-25  206       s->pcm_channels = pcm_channels;
e84d15f6 Clemens Ladisch  2011-09-04  207       s->sfc = sfc;

---
0-DAY kernel build testing backend              Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild                 Intel Corporation
_______________________________________________
kbuild mailing list
kbuild@lists.01.org
https://lists.01.org/mailman/listinfo/kbuild
_______________________________________________
kbuild mailing list
kbuild@lists.01.org
https://lists.01.org/mailman/listinfo/kbuild

Reply via email to