* [PATCH] ALSA: usb-audio: Reorder snd_djm_devices entries @ 2021-12-02 20:55 Geraldo Nascimento 2021-12-03 13:37 ` Takashi Iwai 0 siblings, 1 reply; 3+ messages in thread From: Geraldo Nascimento @ 2021-12-02 20:55 UTC (permalink / raw) To: Takashi Iwai; +Cc: William Overton, ALSA-devel, Olivia Mackintosh 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> diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c index d489c1de3bae..d4cd880b5c6c 100644 --- a/sound/usb/mixer_quirks.c +++ b/sound/usb/mixer_quirks.c @@ -3018,9 +3018,9 @@ 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_DEVICE(900nxs2), + SND_DJM_DEVICE(750mk2) }; ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ALSA: usb-audio: Reorder snd_djm_devices entries 2021-12-02 20:55 [PATCH] ALSA: usb-audio: Reorder snd_djm_devices entries Geraldo Nascimento @ 2021-12-03 13:37 ` Takashi Iwai 2021-12-03 23:58 ` Geraldo Nascimento 0 siblings, 1 reply; 3+ messages in thread From: Takashi Iwai @ 2021-12-03 13:37 UTC (permalink / raw) To: Geraldo Nascimento; +Cc: William Overton, ALSA-devel, Olivia Mackintosh 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> Thanks the patch. The code change is OK, but better to use more explicit form in C99 style initialization, something like below. Could you check whether this works? 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), }; ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ALSA: usb-audio: Reorder snd_djm_devices entries 2021-12-03 13:37 ` Takashi Iwai @ 2021-12-03 23:58 ` Geraldo Nascimento 0 siblings, 0 replies; 3+ messages in thread From: Geraldo Nascimento @ 2021-12-03 23:58 UTC (permalink / raw) To: Takashi Iwai; +Cc: William Overton, ALSA-devel, Olivia Mackintosh 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), > }; > > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-12-04 0:00 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-12-02 20:55 [PATCH] ALSA: usb-audio: Reorder snd_djm_devices entries Geraldo Nascimento 2021-12-03 13:37 ` Takashi Iwai 2021-12-03 23:58 ` Geraldo Nascimento
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.