[PATCH] ALSA: usb-audio: Add mixer support for Pioneer DJ DJM-250MK2

Takashi Iwai tiwai at suse.de
Mon Sep 21 18:36:37 CEST 2020


On Mon, 21 Sep 2020 18:16:12 +0200,
František Kučera wrote:
> 
> From: František Kučera <franta-linux at frantovo.cz>
> 
> This patch extends support for DJM-250MK2 and allows mapping
> playback and capture channels to available sources.
> Configures the card through USB commands.

Could you run scripts/checkpatch.pl, fix the suggested items and
resubmit?


thanks,

Takashi


> ---
>  sound/usb/mixer_quirks.c | 194 +++++++++++++++++++++++++++++++++++++++
>  1 file changed, 194 insertions(+)
> 
> diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c
> index 199cdbfdc761..f2aeb5ac3611 100644
> --- a/sound/usb/mixer_quirks.c
> +++ b/sound/usb/mixer_quirks.c
> @@ -2602,6 +2602,197 @@ static int snd_bbfpro_controls_create(struct usb_mixer_interface *mixer)
>  	return 0;
>  }
>  
> +/*
> + * Pioneer DJ DJM-250MK2 and maybe other DJM models
> + * 
> + * For playback, no duplicate mapping should be set.
> + * There are three mixer stereo channels (CH1, CH2, AUX) and three stereo sources (PLAYBACK 1-2, PLAYBACK 3-4, PLAYBACK 5-6).
> + * Each channel should be mapped just once to one source.
> + * If mapped multiple times, only one source will play on given channel (sources are not mixed together).
> + * 
> + * For recording, duplicate mapping is OK. We will get the same signal multiple times.
> + * 
> + * Channels 7-8 are in both directions fixed to FX SEND / FX RETURN.
> + * 
> + * See also notes in the quirks-table.h file.
> + */
> +
> +typedef struct {
> +	const u16 wIndex;
> +	const u16 wValue;
> +	const char *name;
> +} snd_pioneer_djm_option;
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_CAPTURE_LEVEL[] = {
> +	{ .name =  "-5 dB",                  .wValue = 0x0300, .wIndex = 0x8003 },
> +	{ .name = "-10 dB",                  .wValue = 0x0200, .wIndex = 0x8003 },
> +	{ .name = "-15 dB",                  .wValue = 0x0100, .wIndex = 0x8003 },
> +	{ .name = "-19 dB",                  .wValue = 0x0000, .wIndex = 0x8003 }
> +};
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_CAPTURE_CH12[] = {
> +	{ .name =  "CH1 Control Tone PHONO", .wValue = 0x0103, .wIndex = 0x8002 },
> +	{ .name =  "CH1 Control Tone LINE",  .wValue = 0x0100, .wIndex = 0x8002 },
> +	{ .name =  "Post CH1 Fader",         .wValue = 0x0106, .wIndex = 0x8002 },
> +	{ .name =  "Cross Fader A",          .wValue = 0x0107, .wIndex = 0x8002 },
> +	{ .name =  "Cross Fader B",          .wValue = 0x0108, .wIndex = 0x8002 },
> +	{ .name =  "MIC",                    .wValue = 0x0109, .wIndex = 0x8002 },
> +	{ .name =  "AUX",                    .wValue = 0x010d, .wIndex = 0x8002 },
> +	{ .name =  "REC OUT",                .wValue = 0x010a, .wIndex = 0x8002 }
> +};
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_CAPTURE_CH34[] = {
> +	{ .name =  "CH2 Control Tone PHONO", .wValue = 0x0203, .wIndex = 0x8002 },
> +	{ .name =  "CH2 Control Tone LINE",  .wValue = 0x0200, .wIndex = 0x8002 },
> +	{ .name =  "Post CH2 Fader",         .wValue = 0x0206, .wIndex = 0x8002 },
> +	{ .name =  "Cross Fader A",          .wValue = 0x0207, .wIndex = 0x8002 },
> +	{ .name =  "Cross Fader B",          .wValue = 0x0208, .wIndex = 0x8002 },
> +	{ .name =  "MIC",                    .wValue = 0x0209, .wIndex = 0x8002 },
> +	{ .name =  "AUX",                    .wValue = 0x020d, .wIndex = 0x8002 },
> +	{ .name =  "REC OUT",                .wValue = 0x020a, .wIndex = 0x8002 }
> +};
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_CAPTURE_CH56[] = {
> +	{ .name =  "REC OUT",                .wValue = 0x030a, .wIndex = 0x8002 },
> +	{ .name =  "Post CH1 Fader",         .wValue = 0x0311, .wIndex = 0x8002 },
> +	{ .name =  "Post CH2 Fader",         .wValue = 0x0312, .wIndex = 0x8002 },
> +	{ .name =  "Cross Fader A",          .wValue = 0x0307, .wIndex = 0x8002 },
> +	{ .name =  "Cross Fader B",          .wValue = 0x0308, .wIndex = 0x8002 },
> +	{ .name =  "MIC",                    .wValue = 0x0309, .wIndex = 0x8002 },
> +	{ .name =  "AUX",                    .wValue = 0x030d, .wIndex = 0x8002 }
> +};
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_PLAYBACK_12[] = {
> +	{ .name =  "CH1",                    .wValue = 0x0100, .wIndex = 0x8016 },
> +	{ .name =  "CH2",                    .wValue = 0x0101, .wIndex = 0x8016 },
> +	{ .name =  "AUX",                    .wValue = 0x0104, .wIndex = 0x8016 }
> +};
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_PLAYBACK_34[] = {
> +	{ .name =  "CH1",                    .wValue = 0x0200, .wIndex = 0x8016 },
> +	{ .name =  "CH2",                    .wValue = 0x0201, .wIndex = 0x8016 },
> +	{ .name =  "AUX",                    .wValue = 0x0204, .wIndex = 0x8016 }
> +};
> +
> +static const snd_pioneer_djm_option SND_PIONEER_DJM_OPTIONS_PLAYBACK_56[] = {
> +	{ .name =  "CH1",                    .wValue = 0x0300, .wIndex = 0x8016 },
> +	{ .name =  "CH2",                    .wValue = 0x0301, .wIndex = 0x8016 },
> +	{ .name =  "AUX",                    .wValue = 0x0304, .wIndex = 0x8016 }
> +};
> +
> +typedef struct {
> +	const char *name;
> +	const snd_pioneer_djm_option *options;
> +	const size_t count;
> +	const u16 default_value;
> +} snd_pioneer_djm_option_group;
> +
> +#define SND_PIONEER_DJM_OPTION_GROUP_ITEM(_name, suffix, _default_value) \
> +	{ .name = _name, .options = SND_PIONEER_DJM_OPTIONS_##suffix, .count = ARRAY_SIZE(SND_PIONEER_DJM_OPTIONS_##suffix), .default_value = _default_value }
> +
> +static const snd_pioneer_djm_option_group SND_PIONEER_DJM_OPTION_GROUPS[] = {
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Master Capture Level Capture Switch",  CAPTURE_LEVEL,  0),
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Capture 1-2 Capture Switch",           CAPTURE_CH12,   2),
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Capture 3-4 Capture Switch",           CAPTURE_CH34,   2),
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Capture 5-6 Capture Switch",           CAPTURE_CH56,   0),
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Playback 1-2 Playback Switch",         PLAYBACK_12,    0),
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Playback 3-4 Playback Switch",         PLAYBACK_34,    1),
> +	SND_PIONEER_DJM_OPTION_GROUP_ITEM("Playback 5-6 Playback Switch",         PLAYBACK_56,    2)
> +};
> +
> +// layout of the kcontrol->private_value:
> +#define SND_PIONEER_DJM_VALUE_MASK 0x0000ffff
> +#define SND_PIONEER_DJM_GROUP_MASK 0xffff0000
> +#define SND_PIONEER_DJM_GROUP_SHIFT 16
> +
> +static int snd_pioneer_djm_controls_info(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_info *info) 
> +{
> +	u16 group = kcontrol->private_value >> SND_PIONEER_DJM_GROUP_SHIFT;
> +	size_t count;
> +	const char *name;
> +	if (group < ARRAY_SIZE(SND_PIONEER_DJM_OPTION_GROUPS)) {
> +		count = SND_PIONEER_DJM_OPTION_GROUPS[group].count;
> +		if (info->value.enumerated.item >= count) info->value.enumerated.item = count - 1;
> +		name = SND_PIONEER_DJM_OPTION_GROUPS[group].options[info->value.enumerated.item].name;
> +		strlcpy(info->value.enumerated.name, name, sizeof (info->value.enumerated.name));
> +		info->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED;
> +		info->count = 1;
> +		info->value.enumerated.items = count;
> +		return 0;
> +	} else {
> +		return -EINVAL;
> +	}
> +}
> +
> +static int snd_pioneer_djm_controls_update(struct usb_mixer_interface *mixer, u16 group, u16 value)
> +{
> +	int err;
> +	if (group < ARRAY_SIZE(SND_PIONEER_DJM_OPTION_GROUPS) && value < SND_PIONEER_DJM_OPTION_GROUPS[group].count) {
> +		err = snd_usb_lock_shutdown(mixer->chip);
> +		if (err) return err;
> +		
> +		err = snd_usb_ctl_msg(
> +			mixer->chip->dev, usb_sndctrlpipe(mixer->chip->dev, 0),
> +			USB_REQ_SET_FEATURE,
> +			USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
> +			SND_PIONEER_DJM_OPTION_GROUPS[group].options[value].wValue,
> +			SND_PIONEER_DJM_OPTION_GROUPS[group].options[value].wIndex,
> +			NULL, 0);
> +		
> +		snd_usb_unlock_shutdown(mixer->chip);
> +		return err;
> +	} else {
> +		return -EINVAL;
> +	}
> +}
> +
> +static int snd_pioneer_djm_controls_get(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol)
> +{
> +	ucontrol->value.enumerated.item[0] = kcontrol->private_value & SND_PIONEER_DJM_VALUE_MASK;
> +	return 0;
> +}
> +
> +static int snd_pioneer_djm_controls_put(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol)
> +{
> +	struct usb_mixer_elem_list *list = snd_kcontrol_chip(kcontrol);
> +	struct usb_mixer_interface *mixer = list->mixer;
> +	
> +	u16 group = (kcontrol->private_value & SND_PIONEER_DJM_GROUP_MASK) >> SND_PIONEER_DJM_GROUP_SHIFT;
> +	u16 value = ucontrol->value.enumerated.item[0];
> +	kcontrol->private_value = group << SND_PIONEER_DJM_GROUP_SHIFT | value;
> +	
> +	return snd_pioneer_djm_controls_update(mixer, group, value);
> +}
> +
> +static int snd_pioneer_djm_controls_resume(struct usb_mixer_elem_list *list)
> +{
> +	u16 group = (list->kctl->private_value & SND_PIONEER_DJM_GROUP_MASK) >> SND_PIONEER_DJM_GROUP_SHIFT;
> +	u16 value = (list->kctl->private_value & SND_PIONEER_DJM_VALUE_MASK);
> +	return snd_pioneer_djm_controls_update(list->mixer, group, value);
> +}
> +
> +static int snd_pioneer_djm_controls_create(struct usb_mixer_interface *mixer)
> +{
> +	int err, group;
> +	for (group = 0; group < ARRAY_SIZE(SND_PIONEER_DJM_OPTION_GROUPS); group++ ) {
> +		struct snd_kcontrol_new knew = {
> +			.iface  = SNDRV_CTL_ELEM_IFACE_MIXER,
> +			.access = SNDRV_CTL_ELEM_ACCESS_READWRITE,
> +			.index = 0,
> +			.info = snd_pioneer_djm_controls_info,
> +			.get  = snd_pioneer_djm_controls_get,
> +			.put  = snd_pioneer_djm_controls_put,
> +			.name = SND_PIONEER_DJM_OPTION_GROUPS[group].name,
> +			.private_value = group << SND_PIONEER_DJM_GROUP_SHIFT | SND_PIONEER_DJM_OPTION_GROUPS[group].default_value
> +		};
> +		err = snd_pioneer_djm_controls_update(mixer, group, SND_PIONEER_DJM_OPTION_GROUPS[group].default_value);
> +		if (err) return err;
> +		err = add_single_ctl_with_resume(mixer, 0, snd_pioneer_djm_controls_resume, &knew, NULL);
> +		if (err) return err;
> +	}
> +	return 0;
> +}
> +
>  int snd_usb_mixer_apply_create_quirk(struct usb_mixer_interface *mixer)
>  {
>  	int err = 0;
> @@ -2706,6 +2897,9 @@ int snd_usb_mixer_apply_create_quirk(struct usb_mixer_interface *mixer)
>  	case USB_ID(0x2a39, 0x3fb0): /* RME Babyface Pro FS */
>  		err = snd_bbfpro_controls_create(mixer);
>  		break;
> +	case USB_ID(0x2b73, 0x0017): /* Pioneer DJ DJM-250MK2 */
> +		err = snd_pioneer_djm_controls_create(mixer);
> +		break;
>  	}
>  
>  	return err;
> -- 
> 2.20.1
> 


More information about the Alsa-devel mailing list