* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).