Re: [PATCH v2] ALSA: usb-audio: Reorder snd_djm_devices[] entries

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 04 Dec 2021 19:52:24 +0100,
Geraldo Nascimento wrote:
> 
> Olivia Mackintosh has posted to alsa-devel reporting that
> there's a potential bug that could break mixer quirks for Pioneer
> devices introduced by 6d27788160362a7ee6c0d317636fe4b1ddbe59a7
> "ALSA: usb-audio: Add support for the Pioneer DJM 750MK2
> Mixer/Soundcard".
> 
> This happened because the DJM 750 MK2 was added last to the Pioneer DJM
> device table index and defined as 0x4 but was added to snd_djm_devices[]
> just after the DJM 750 (MK1) entry instead of last, after the DJM 900
> NXS2. This escaped review.
> 
> To prevent that from ever happening again, Takashi Iwai suggested to use
> C99 array designators in snd_djm_devices[] instead of simply reordering
> the entries.
> 
> Reported-by: Olivia Mackintosh <livvy@xxxxxxx>
> Suggested-by: Takashi Iwai <tiwai@xxxxxxx>
> Signed-off-by: Geraldo Nascimento <geraldogabriel@xxxxxxxxx>
> 
> ---
> 
> V1 -> V2: added C99 array designators to prevent the order of entries
> ever being an issue again.

Thanks, applied now with Fixes tag.


Takashi



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux