ALSA: firewire-motu: suppless consumption for unused element of array in stack
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Mon, 18 Jun 2018 12:07:51 +0000 (21:07 +0900)
committerTakashi Iwai <tiwai@suse.de>
Tue, 19 Jun 2018 18:44:45 +0000 (20:44 +0200)
In MOTU firewire protocol, data block consists of 24 bit data chunks except
for one quadlet for source packet header (SPH). The number of data chunk in
a data block is different between three clock modes; low, middle and high.
When unit supports ADAT on optical interface, the data block includes some
chunks for ADAT channels. These ADAT chunks are unavailable at high mode.

This driver has local functions to calculate the number of ADAT chunks. But
They uses stack for three clock modes. This is useless for higher mode.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/motu/motu-protocol-v2.c

index 525b746..a51fd19 100644 (file)
@@ -176,7 +176,7 @@ static void calculate_differed_part(struct snd_motu_packet_format *formats,
                                    enum snd_motu_spec_flags flags,
                                    u32 data, u32 mask, u32 shift)
 {
-       unsigned char pcm_chunks[3] = {0, 0};
+       unsigned char pcm_chunks[2] = {0, 0};
 
        /*
         * When optical interfaces are configured for S/PDIF (TOSLINK),