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

[Geraldo Nascimento <geraldogabriel@gmail.com>]

On Fri, Dec 03, 2021 at 02:37:07PM +0100, Takashi Iwai wrote:
> On Thu, 02 Dec 2021 21:55:51 +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.
> > 
> > Let's reorder the entries in snd_djm_devices[] so they match the Pioneer
> > DJM device table index #define's.
> > 
> > Reported-by: Olivia Mackintosh <livvy@base.nu>
> > Signed-off-by: Geraldo Nascimento <geraldogabriel@gmail.com>
>

Hi, Takashi

> Thanks the patch.

My pleasure.

> 
> The code change is OK, but better to use more explicit form in C99
> style initialization, something like below.

I'll send v2 of the patch with C99-style designators as you have
suggested.

> 
> Could you check whether this works?

That's the problem, I don't own any DJM mixer, someone else needs to
test this.

Thanks,
Geraldo Nascimento

> 
> 
> thanks,
> 
> Takashi
> 
> --- a/sound/usb/mixer_quirks.c
> +++ b/sound/usb/mixer_quirks.c
> @@ -3016,11 +3016,11 @@ static const struct snd_djm_ctl snd_djm_ctls_750mk2[] = {
>  
>  
>  static const struct snd_djm_device snd_djm_devices[] = {
> -	SND_DJM_DEVICE(250mk2),
> -	SND_DJM_DEVICE(750),
> -	SND_DJM_DEVICE(750mk2),
> -	SND_DJM_DEVICE(850),
> -	SND_DJM_DEVICE(900nxs2)
> +	[SND_DJM_250MK2_IDX] =	SND_DJM_DEVICE(250mk2),
> +	[SND_DJM_750_IDX] =	SND_DJM_DEVICE(750),
> +	[SND_DJM_850_IDX] =	SND_DJM_DEVICE(850),
> +	[SND_DJM_900NXS2_IDX] =	SND_DJM_DEVICE(900nxs2),
> +	[SND_DJM_750MK2_IDX] =	SND_DJM_DEVICE(750mk2),
>  };
>  
>