Re: [PATCH] ALSA: usb-audio: Support multiple control interfaces

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



On Sat, 10 Aug 2024 20:49:55 +0200,
Karol Kosik wrote:
> 
> USB standard allows for a device to have many audio functions, each
> represented by Audio Interface Collection. Every audio function is
> considered closed box and will contain unique AudioControl interface and
> zero or more AudioStreaming and MIDIStreaming interfaces.
> 
> Add structure holding association between each interface and its
> AudioControl Interface counterpart, instead of relying on AudioControl
> Interface define for the entire device.
> 
> Signed-off-by: Karol Kosik <k.kosik@xxxxxxxxxxx>

Thanks for the patch.  As it's an intensive change, I'll have a closer
look tomorrow.  But, before that: could you improve the patch
description?

I suppose it's a fix corresponding to the kernel bugzilla entry,
right?
  https://bugzilla.kernel.org/show_bug.cgi?id=217865
If so, please put the reference as a "Link:" tag.  And, begin with the
problem statement (a Numark device doesn't work), followed by your
analysis of the cause (the device having multiple AudioControl
interfaces while the driver blindly assumes only the first one).
The patch description you included in this patch is applied after
that.

And you don't have to save the space for more explanations.
You can put the details you've put in Bugzilla comment to here, too
(including an alternative solution and why you chose this way).

Last but not least, it'd be worth to mention about the influence to
the other devices.  There are literally hundreds of different models
covered by this driver, and we shouldn't introduce a regression on all
of those.


thanks,

Takashi

> ---
>  sound/usb/card.c           |  2 ++
>  sound/usb/clock.c          | 62 ++++++++++++++++++++++++--------------
>  sound/usb/format.c         |  6 ++--
>  sound/usb/helper.c         | 34 +++++++++++++++++++++
>  sound/usb/helper.h         | 10 ++++--
>  sound/usb/mixer.c          |  2 +-
>  sound/usb/mixer_quirks.c   | 17 ++++++-----
>  sound/usb/mixer_scarlett.c |  4 +--
>  sound/usb/power.c          |  3 +-
>  sound/usb/power.h          |  1 +
>  sound/usb/stream.c         | 21 ++++++++-----
>  sound/usb/usbaudio.h       | 12 ++++++++
>  12 files changed, 127 insertions(+), 47 deletions(-)
> 
> diff --git a/sound/usb/card.c b/sound/usb/card.c
> index bdb04fa37a71..778de9244f1e 100644
> --- a/sound/usb/card.c
> +++ b/sound/usb/card.c
> @@ -206,6 +206,8 @@ static int snd_usb_create_stream(struct snd_usb_audio *chip, int ctrlif, int int
>  		return -EINVAL;
>  	}
>  
> +	snd_usb_add_ctrl_interface_link(chip, interface, ctrlif);
> +
>  	if (! snd_usb_parse_audio_interface(chip, interface)) {
>  		usb_set_interface(dev, interface, 0); /* reset the current interface */
>  		return usb_driver_claim_interface(&usb_audio_driver, iface,
> diff --git a/sound/usb/clock.c b/sound/usb/clock.c
> index 60fcb872a80b..8f85200292f3 100644
> --- a/sound/usb/clock.c
> +++ b/sound/usb/clock.c
> @@ -76,11 +76,14 @@ static bool validate_clock_multiplier(void *p, int id, int proto)
>  }
>  
>  #define DEFINE_FIND_HELPER(name, obj, validator, type2, type3)		\
> -static obj *name(struct snd_usb_audio *chip, int id, int proto)	\
> +static obj *name(struct snd_usb_audio *chip, int id,	\
> +				const struct audioformat *fmt)	\
>  {									\
> -	return find_uac_clock_desc(chip->ctrl_intf, id, validator,	\
> -				   proto == UAC_VERSION_3 ? (type3) : (type2), \
> -				   proto);				\
> +	struct usb_host_interface *ctrl_intf =	\
> +		snd_usb_find_ctrl_interface(chip, fmt->iface); \
> +	return find_uac_clock_desc(ctrl_intf, id, validator,	\
> +				   fmt->protocol == UAC_VERSION_3 ? (type3) : (type2), \
> +				   fmt->protocol);				\
>  }
>  
>  DEFINE_FIND_HELPER(snd_usb_find_clock_source,
> @@ -93,16 +96,19 @@ DEFINE_FIND_HELPER(snd_usb_find_clock_multiplier,
>  		   union uac23_clock_multiplier_desc, validate_clock_multiplier,
>  		   UAC2_CLOCK_MULTIPLIER, UAC3_CLOCK_MULTIPLIER);
>  
> -static int uac_clock_selector_get_val(struct snd_usb_audio *chip, int selector_id)
> +static int uac_clock_selector_get_val(struct snd_usb_audio *chip,
> +				int selector_id, int iface_no)
>  {
> +	struct usb_host_interface *ctrl_intf;
>  	unsigned char buf;
>  	int ret;
>  
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, iface_no);
>  	ret = snd_usb_ctl_msg(chip->dev, usb_rcvctrlpipe(chip->dev, 0),
>  			      UAC2_CS_CUR,
>  			      USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_IN,
>  			      UAC2_CX_CLOCK_SELECTOR << 8,
> -			      snd_usb_ctrl_intf(chip) | (selector_id << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (selector_id << 8),
>  			      &buf, sizeof(buf));
>  
>  	if (ret < 0)
> @@ -111,16 +117,18 @@ static int uac_clock_selector_get_val(struct snd_usb_audio *chip, int selector_i
>  	return buf;
>  }
>  
> -static int uac_clock_selector_set_val(struct snd_usb_audio *chip, int selector_id,
> -					unsigned char pin)
> +static int uac_clock_selector_set_val(struct snd_usb_audio *chip,
> +					int selector_id, unsigned char pin, int iface_no)
>  {
> +	struct usb_host_interface *ctrl_intf;
>  	int ret;
>  
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, iface_no);
>  	ret = snd_usb_ctl_msg(chip->dev, usb_sndctrlpipe(chip->dev, 0),
>  			      UAC2_CS_CUR,
>  			      USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_OUT,
>  			      UAC2_CX_CLOCK_SELECTOR << 8,
> -			      snd_usb_ctrl_intf(chip) | (selector_id << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (selector_id << 8),
>  			      &pin, sizeof(pin));
>  	if (ret < 0)
>  		return ret;
> @@ -132,7 +140,7 @@ static int uac_clock_selector_set_val(struct snd_usb_audio *chip, int selector_i
>  		return -EINVAL;
>  	}
>  
> -	ret = uac_clock_selector_get_val(chip, selector_id);
> +	ret = uac_clock_selector_get_val(chip, selector_id, iface_no);
>  	if (ret < 0)
>  		return ret;
>  
> @@ -155,8 +163,10 @@ static bool uac_clock_source_is_valid_quirk(struct snd_usb_audio *chip,
>  	unsigned char data;
>  	struct usb_device *dev = chip->dev;
>  	union uac23_clock_source_desc *cs_desc;
> +	struct usb_host_interface *ctrl_intf;
>  
> -	cs_desc = snd_usb_find_clock_source(chip, source_id, fmt->protocol);
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, fmt->iface);
> +	cs_desc = snd_usb_find_clock_source(chip, source_id, fmt);
>  	if (!cs_desc)
>  		return false;
>  
> @@ -191,7 +201,7 @@ static bool uac_clock_source_is_valid_quirk(struct snd_usb_audio *chip,
>  			err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_CUR,
>  					      USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
>  					      UAC2_CS_CONTROL_CLOCK_VALID << 8,
> -					      snd_usb_ctrl_intf(chip) | (source_id << 8),
> +					      snd_usb_ctrl_intf(ctrl_intf) | (source_id << 8),
>  					      &data, sizeof(data));
>  			if (err < 0) {
>  				dev_warn(&dev->dev,
> @@ -217,8 +227,10 @@ static bool uac_clock_source_is_valid(struct snd_usb_audio *chip,
>  	struct usb_device *dev = chip->dev;
>  	u32 bmControls;
>  	union uac23_clock_source_desc *cs_desc;
> +	struct usb_host_interface *ctrl_intf;
>  
> -	cs_desc = snd_usb_find_clock_source(chip, source_id, fmt->protocol);
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, fmt->iface);
> +	cs_desc = snd_usb_find_clock_source(chip, source_id, fmt);
>  	if (!cs_desc)
>  		return false;
>  
> @@ -235,7 +247,7 @@ static bool uac_clock_source_is_valid(struct snd_usb_audio *chip,
>  	err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_CUR,
>  			      USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
>  			      UAC2_CS_CONTROL_CLOCK_VALID << 8,
> -			      snd_usb_ctrl_intf(chip) | (source_id << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (source_id << 8),
>  			      &data, sizeof(data));
>  
>  	if (err < 0) {
> @@ -274,7 +286,7 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip,
>  	}
>  
>  	/* first, see if the ID we're looking at is a clock source already */
> -	source = snd_usb_find_clock_source(chip, entity_id, proto);
> +	source = snd_usb_find_clock_source(chip, entity_id, fmt);
>  	if (source) {
>  		entity_id = GET_VAL(source, proto, bClockID);
>  		if (validate && !uac_clock_source_is_valid(chip, fmt,
> @@ -287,7 +299,7 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip,
>  		return entity_id;
>  	}
>  
> -	selector = snd_usb_find_clock_selector(chip, entity_id, proto);
> +	selector = snd_usb_find_clock_selector(chip, entity_id, fmt);
>  	if (selector) {
>  		pins = GET_VAL(selector, proto, bNrInPins);
>  		clock_id = GET_VAL(selector, proto, bClockID);
> @@ -317,7 +329,7 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip,
>  
>  		/* the entity ID we are looking at is a selector.
>  		 * find out what it currently selects */
> -		ret = uac_clock_selector_get_val(chip, clock_id);
> +		ret = uac_clock_selector_get_val(chip, clock_id, fmt->iface);
>  		if (ret < 0) {
>  			if (!chip->autoclock)
>  				return ret;
> @@ -346,7 +358,7 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip,
>  			if (chip->quirk_flags & QUIRK_FLAG_SKIP_CLOCK_SELECTOR ||
>  			    !writeable)
>  				return ret;
> -			err = uac_clock_selector_set_val(chip, entity_id, cur);
> +			err = uac_clock_selector_set_val(chip, entity_id, cur, fmt->iface);
>  			if (err < 0) {
>  				if (pins == 1) {
>  					usb_audio_dbg(chip,
> @@ -377,7 +389,7 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip,
>  			if (ret < 0)
>  				continue;
>  
> -			err = uac_clock_selector_set_val(chip, entity_id, i);
> +			err = uac_clock_selector_set_val(chip, entity_id, i, fmt->iface);
>  			if (err < 0)
>  				continue;
>  
> @@ -391,7 +403,7 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip,
>  	}
>  
>  	/* FIXME: multipliers only act as pass-thru element for now */
> -	multiplier = snd_usb_find_clock_multiplier(chip, entity_id, proto);
> +	multiplier = snd_usb_find_clock_multiplier(chip, entity_id, fmt);
>  	if (multiplier)
>  		return __uac_clock_find_source(chip, fmt,
>  					       GET_VAL(multiplier, proto, bCSourceID),
> @@ -491,11 +503,13 @@ static int get_sample_rate_v2v3(struct snd_usb_audio *chip, int iface,
>  	struct usb_device *dev = chip->dev;
>  	__le32 data;
>  	int err;
> +	struct usb_host_interface *ctrl_intf;
>  
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, iface);
>  	err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_CUR,
>  			      USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
>  			      UAC2_CS_CONTROL_SAM_FREQ << 8,
> -			      snd_usb_ctrl_intf(chip) | (clock << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (clock << 8),
>  			      &data, sizeof(data));
>  	if (err < 0) {
>  		dev_warn(&dev->dev, "%d:%d: cannot get freq (v2/v3): err %d\n",
> @@ -524,8 +538,10 @@ int snd_usb_set_sample_rate_v2v3(struct snd_usb_audio *chip,
>  	__le32 data;
>  	int err;
>  	union uac23_clock_source_desc *cs_desc;
> +	struct usb_host_interface *ctrl_intf;
>  
> -	cs_desc = snd_usb_find_clock_source(chip, clock, fmt->protocol);
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, fmt->iface);
> +	cs_desc = snd_usb_find_clock_source(chip, clock, fmt);
>  
>  	if (!cs_desc)
>  		return 0;
> @@ -544,7 +560,7 @@ int snd_usb_set_sample_rate_v2v3(struct snd_usb_audio *chip,
>  	err = snd_usb_ctl_msg(chip->dev, usb_sndctrlpipe(chip->dev, 0), UAC2_CS_CUR,
>  			      USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_OUT,
>  			      UAC2_CS_CONTROL_SAM_FREQ << 8,
> -			      snd_usb_ctrl_intf(chip) | (clock << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (clock << 8),
>  			      &data, sizeof(data));
>  	if (err < 0)
>  		return err;
> diff --git a/sound/usb/format.c b/sound/usb/format.c
> index 1bb6a455a1b4..0cbf1d4fbe6e 100644
> --- a/sound/usb/format.c
> +++ b/sound/usb/format.c
> @@ -545,7 +545,9 @@ static int parse_audio_format_rates_v2v3(struct snd_usb_audio *chip,
>  	unsigned char tmp[2], *data;
>  	int nr_triplets, data_size, ret = 0, ret_l6;
>  	int clock = snd_usb_clock_find_source(chip, fp, false);
> +	struct usb_host_interface *ctrl_intf;
>  
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, fp->iface);
>  	if (clock < 0) {
>  		dev_err(&dev->dev,
>  			"%s(): unable to find clock source (clock %d)\n",
> @@ -557,7 +559,7 @@ static int parse_audio_format_rates_v2v3(struct snd_usb_audio *chip,
>  	ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_RANGE,
>  			      USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
>  			      UAC2_CS_CONTROL_SAM_FREQ << 8,
> -			      snd_usb_ctrl_intf(chip) | (clock << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (clock << 8),
>  			      tmp, sizeof(tmp));
>  
>  	if (ret < 0) {
> @@ -592,7 +594,7 @@ static int parse_audio_format_rates_v2v3(struct snd_usb_audio *chip,
>  	ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_RANGE,
>  			      USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
>  			      UAC2_CS_CONTROL_SAM_FREQ << 8,
> -			      snd_usb_ctrl_intf(chip) | (clock << 8),
> +			      snd_usb_ctrl_intf(ctrl_intf) | (clock << 8),
>  			      data, data_size);
>  
>  	if (ret < 0) {
> diff --git a/sound/usb/helper.c b/sound/usb/helper.c
> index bf80e55d013a..72b671fb2c84 100644
> --- a/sound/usb/helper.c
> +++ b/sound/usb/helper.c
> @@ -130,3 +130,37 @@ snd_usb_get_host_interface(struct snd_usb_audio *chip, int ifnum, int altsetting
>  		return NULL;
>  	return usb_altnum_to_altsetting(iface, altsetting);
>  }
> +
> +int snd_usb_add_ctrl_interface_link(struct snd_usb_audio *chip, int ifnum,
> +		int ctrlif)
> +{
> +	struct usb_device *dev = chip->dev;
> +	struct usb_host_interface *host_iface;
> +
> +	if (chip->num_intf_to_ctrl >= MAX_CARD_INTERFACES) {
> +		dev_info(&dev->dev, "Too many interfaces assigned to the single USB-audio card\n");
> +		return -EINVAL;
> +	}
> +
> +	/* find audiocontrol interface */
> +	host_iface = &usb_ifnum_to_if(dev, ctrlif)->altsetting[0];
> +
> +	chip->intf_to_ctrl[chip->num_intf_to_ctrl].interface = ifnum;
> +	chip->intf_to_ctrl[chip->num_intf_to_ctrl].ctrl_intf = host_iface;
> +	chip->num_intf_to_ctrl++;
> +
> +	return 0;
> +}
> +
> +struct usb_host_interface *snd_usb_find_ctrl_interface(struct snd_usb_audio *chip,
> +							int ifnum)
> +{
> +	int i;
> +
> +	for (i = 0; i < chip->num_intf_to_ctrl; ++i)
> +		if (chip->intf_to_ctrl[i].interface == ifnum)
> +			return chip->intf_to_ctrl[i].ctrl_intf;
> +
> +	/* Fallback to first audiocontrol interface */
> +	return chip->ctrl_intf;
> +}
> diff --git a/sound/usb/helper.h b/sound/usb/helper.h
> index e2b51ec96ec6..0372e050b3dc 100644
> --- a/sound/usb/helper.h
> +++ b/sound/usb/helper.h
> @@ -17,6 +17,12 @@ unsigned char snd_usb_parse_datainterval(struct snd_usb_audio *chip,
>  struct usb_host_interface *
>  snd_usb_get_host_interface(struct snd_usb_audio *chip, int ifnum, int altsetting);
>  
> +int snd_usb_add_ctrl_interface_link(struct snd_usb_audio *chip, int ifnum,
> +		int ctrlif);
> +
> +struct usb_host_interface *snd_usb_find_ctrl_interface(struct snd_usb_audio *chip,
> +								int ifnum);
> +
>  /*
>   * retrieve usb_interface descriptor from the host interface
>   * (conditional for compatibility with the older API)
> @@ -28,9 +34,9 @@ snd_usb_get_host_interface(struct snd_usb_audio *chip, int ifnum, int altsetting
>  
>  #define snd_usb_get_speed(dev) ((dev)->speed)
>  
> -static inline int snd_usb_ctrl_intf(struct snd_usb_audio *chip)
> +static inline int snd_usb_ctrl_intf(struct usb_host_interface *ctrl_intf)
>  {
> -	return get_iface_desc(chip->ctrl_intf)->bInterfaceNumber;
> +	return get_iface_desc(ctrl_intf)->bInterfaceNumber;
>  }
>  
>  /* in validate.c */
> diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c
> index 2d27d729c3be..9945ae55b0d0 100644
> --- a/sound/usb/mixer.c
> +++ b/sound/usb/mixer.c
> @@ -728,7 +728,7 @@ static int get_cluster_channels_v3(struct mixer_build *state, unsigned int clust
>  			UAC3_CS_REQ_HIGH_CAPABILITY_DESCRIPTOR,
>  			USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_IN,
>  			cluster_id,
> -			snd_usb_ctrl_intf(state->chip),
> +			snd_usb_ctrl_intf(state->mixer->hostif),
>  			&c_header, sizeof(c_header));
>  	if (err < 0)
>  		goto error;
> diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c
> index d56e76fd5cbe..5d6792f51f4c 100644
> --- a/sound/usb/mixer_quirks.c
> +++ b/sound/usb/mixer_quirks.c
> @@ -1043,7 +1043,7 @@ static int snd_ftu_eff_switch_init(struct usb_mixer_interface *mixer,
>  	err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC_GET_CUR,
>  			      USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_IN,
>  			      pval & 0xff00,
> -			      snd_usb_ctrl_intf(mixer->chip) | ((pval & 0xff) << 8),
> +			      snd_usb_ctrl_intf(mixer->hostif) | ((pval & 0xff) << 8),
>  			      value, 2);
>  	if (err < 0)
>  		return err;
> @@ -1077,7 +1077,7 @@ static int snd_ftu_eff_switch_update(struct usb_mixer_elem_list *list)
>  			      UAC_SET_CUR,
>  			      USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_OUT,
>  			      pval & 0xff00,
> -			      snd_usb_ctrl_intf(chip) | ((pval & 0xff) << 8),
> +			      snd_usb_ctrl_intf(list->mixer->hostif) | ((pval & 0xff) << 8),
>  			      value, 2);
>  	snd_usb_unlock_shutdown(chip);
>  	return err;
> @@ -2115,24 +2115,25 @@ static int dell_dock_mixer_create(struct usb_mixer_interface *mixer)
>  	return 0;
>  }
>  
> -static void dell_dock_init_vol(struct snd_usb_audio *chip, int ch, int id)
> +static void dell_dock_init_vol(struct usb_mixer_interface *mixer, int ch, int id)
>  {
> +	struct snd_usb_audio *chip = mixer->chip;
>  	u16 buf = 0;
>  
>  	snd_usb_ctl_msg(chip->dev, usb_sndctrlpipe(chip->dev, 0), UAC_SET_CUR,
>  			USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_OUT,
>  			(UAC_FU_VOLUME << 8) | ch,
> -			snd_usb_ctrl_intf(chip) | (id << 8),
> +			snd_usb_ctrl_intf(mixer->hostif) | (id << 8),
>  			&buf, 2);
>  }
>  
>  static int dell_dock_mixer_init(struct usb_mixer_interface *mixer)
>  {
>  	/* fix to 0dB playback volumes */
> -	dell_dock_init_vol(mixer->chip, 1, 16);
> -	dell_dock_init_vol(mixer->chip, 2, 16);
> -	dell_dock_init_vol(mixer->chip, 1, 19);
> -	dell_dock_init_vol(mixer->chip, 2, 19);
> +	dell_dock_init_vol(mixer, 1, 16);
> +	dell_dock_init_vol(mixer, 2, 16);
> +	dell_dock_init_vol(mixer, 1, 19);
> +	dell_dock_init_vol(mixer, 2, 19);
>  	return 0;
>  }
>  
> diff --git a/sound/usb/mixer_scarlett.c b/sound/usb/mixer_scarlett.c
> index 0d6e4f15bf77..ff548041679b 100644
> --- a/sound/usb/mixer_scarlett.c
> +++ b/sound/usb/mixer_scarlett.c
> @@ -460,7 +460,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  	struct snd_usb_audio *chip = elem->head.mixer->chip;
>  	unsigned char buf[2 * MAX_CHANNELS] = {0, };
>  	int wValue = (elem->control << 8) | elem->idx_off;
> -	int idx = snd_usb_ctrl_intf(chip) | (elem->head.id << 8);
> +	int idx = snd_usb_ctrl_intf(elem->head.mixer->hostif) | (elem->head.id << 8);
>  	int err;
>  
>  	err = snd_usb_ctl_msg(chip->dev,
> @@ -1002,7 +1002,7 @@ int snd_scarlett_controls_create(struct usb_mixer_interface *mixer)
>  	err = snd_usb_ctl_msg(mixer->chip->dev,
>  		usb_sndctrlpipe(mixer->chip->dev, 0), UAC2_CS_CUR,
>  		USB_RECIP_INTERFACE | USB_TYPE_CLASS |
> -		USB_DIR_OUT, 0x0100, snd_usb_ctrl_intf(mixer->chip) |
> +		USB_DIR_OUT, 0x0100, snd_usb_ctrl_intf(mixer->hostif) |
>  		(0x29 << 8), sample_rate_buffer, 4);
>  	if (err < 0)
>  		return err;
> diff --git a/sound/usb/power.c b/sound/usb/power.c
> index 606a2cb23eab..66bd4daa68fd 100644
> --- a/sound/usb/power.c
> +++ b/sound/usb/power.c
> @@ -40,6 +40,7 @@ snd_usb_find_power_domain(struct usb_host_interface *ctrl_iface,
>  					le16_to_cpu(pd_desc->waRecoveryTime1);
>  				pd->pd_d2d0_rec =
>  					le16_to_cpu(pd_desc->waRecoveryTime2);
> +				pd->ctrl_iface = ctrl_iface;
>  				return pd;
>  			}
>  		}
> @@ -57,7 +58,7 @@ int snd_usb_power_domain_set(struct snd_usb_audio *chip,
>  	unsigned char current_state;
>  	int err, idx;
>  
> -	idx = snd_usb_ctrl_intf(chip) | (pd->pd_id << 8);
> +	idx = snd_usb_ctrl_intf(pd->ctrl_iface) | (pd->pd_id << 8);
>  
>  	err = snd_usb_ctl_msg(chip->dev, usb_rcvctrlpipe(chip->dev, 0),
>  			      UAC2_CS_CUR,
> diff --git a/sound/usb/power.h b/sound/usb/power.h
> index 396e3e51440a..1fa92ad0ca92 100644
> --- a/sound/usb/power.h
> +++ b/sound/usb/power.h
> @@ -6,6 +6,7 @@ struct snd_usb_power_domain {
>  	int pd_id;              /* UAC3 Power Domain ID */
>  	int pd_d1d0_rec;        /* D1 to D0 recovery time */
>  	int pd_d2d0_rec;        /* D2 to D0 recovery time */
> +	struct usb_host_interface *ctrl_iface; /* Control interface */
>  };
>  
>  enum {
> diff --git a/sound/usb/stream.c b/sound/usb/stream.c
> index e14c725acebf..d70c140813d6 100644
> --- a/sound/usb/stream.c
> +++ b/sound/usb/stream.c
> @@ -713,10 +713,13 @@ snd_usb_get_audioformat_uac12(struct snd_usb_audio *chip,
>  	struct usb_device *dev = chip->dev;
>  	struct uac_format_type_i_continuous_descriptor *fmt;
>  	unsigned int num_channels = 0, chconfig = 0;
> +	struct usb_host_interface *ctrl_intf;
>  	struct audioformat *fp;
>  	int clock = 0;
>  	u64 format;
>  
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, iface_no);
> +
>  	/* get audio formats */
>  	if (protocol == UAC_VERSION_1) {
>  		struct uac1_as_header_descriptor *as =
> @@ -740,7 +743,7 @@ snd_usb_get_audioformat_uac12(struct snd_usb_audio *chip,
>  
>  		format = le16_to_cpu(as->wFormatTag); /* remember the format value */
>  
> -		iterm = snd_usb_find_input_terminal_descriptor(chip->ctrl_intf,
> +		iterm = snd_usb_find_input_terminal_descriptor(ctrl_intf,
>  							       as->bTerminalLink,
>  							       protocol);
>  		if (iterm) {
> @@ -776,7 +779,7 @@ snd_usb_get_audioformat_uac12(struct snd_usb_audio *chip,
>  		 * lookup the terminal associated to this interface
>  		 * to extract the clock
>  		 */
> -		input_term = snd_usb_find_input_terminal_descriptor(chip->ctrl_intf,
> +		input_term = snd_usb_find_input_terminal_descriptor(ctrl_intf,
>  								    as->bTerminalLink,
>  								    protocol);
>  		if (input_term) {
> @@ -786,7 +789,7 @@ snd_usb_get_audioformat_uac12(struct snd_usb_audio *chip,
>  			goto found_clock;
>  		}
>  
> -		output_term = snd_usb_find_output_terminal_descriptor(chip->ctrl_intf,
> +		output_term = snd_usb_find_output_terminal_descriptor(ctrl_intf,
>  								      as->bTerminalLink,
>  								      protocol);
>  		if (output_term) {
> @@ -870,6 +873,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  	struct uac3_cluster_header_descriptor *cluster;
>  	struct uac3_as_header_descriptor *as = NULL;
>  	struct uac3_hc_descriptor_header hc_header;
> +	struct usb_host_interface *ctrl_intf;
>  	struct snd_pcm_chmap_elem *chmap;
>  	struct snd_usb_power_domain *pd;
>  	unsigned char badd_profile;
> @@ -881,6 +885,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  	int err;
>  
>  	badd_profile = chip->badd_profile;
> +	ctrl_intf = snd_usb_find_ctrl_interface(chip, iface_no);
>  
>  	if (badd_profile >= UAC3_FUNCTION_SUBCLASS_GENERIC_IO) {
>  		unsigned int maxpacksize =
> @@ -966,7 +971,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  			UAC3_CS_REQ_HIGH_CAPABILITY_DESCRIPTOR,
>  			USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_IN,
>  			cluster_id,
> -			snd_usb_ctrl_intf(chip),
> +			snd_usb_ctrl_intf(ctrl_intf),
>  			&hc_header, sizeof(hc_header));
>  	if (err < 0)
>  		return ERR_PTR(err);
> @@ -990,7 +995,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  			UAC3_CS_REQ_HIGH_CAPABILITY_DESCRIPTOR,
>  			USB_RECIP_INTERFACE | USB_TYPE_CLASS | USB_DIR_IN,
>  			cluster_id,
> -			snd_usb_ctrl_intf(chip),
> +			snd_usb_ctrl_intf(ctrl_intf),
>  			cluster, wLength);
>  	if (err < 0) {
>  		kfree(cluster);
> @@ -1011,7 +1016,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  	 * lookup the terminal associated to this interface
>  	 * to extract the clock
>  	 */
> -	input_term = snd_usb_find_input_terminal_descriptor(chip->ctrl_intf,
> +	input_term = snd_usb_find_input_terminal_descriptor(ctrl_intf,
>  							    as->bTerminalLink,
>  							    UAC_VERSION_3);
>  	if (input_term) {
> @@ -1019,7 +1024,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  		goto found_clock;
>  	}
>  
> -	output_term = snd_usb_find_output_terminal_descriptor(chip->ctrl_intf,
> +	output_term = snd_usb_find_output_terminal_descriptor(ctrl_intf,
>  							      as->bTerminalLink,
>  							      UAC_VERSION_3);
>  	if (output_term) {
> @@ -1068,7 +1073,7 @@ snd_usb_get_audioformat_uac3(struct snd_usb_audio *chip,
>  							       UAC_VERSION_3,
>  							       iface_no);
>  
> -		pd = snd_usb_find_power_domain(chip->ctrl_intf,
> +		pd = snd_usb_find_power_domain(ctrl_intf,
>  					       as->bTerminalLink);
>  
>  		/* ok, let's parse further... */
> diff --git a/sound/usb/usbaudio.h b/sound/usb/usbaudio.h
> index 43d4029edab4..b0f042c99608 100644
> --- a/sound/usb/usbaudio.h
> +++ b/sound/usb/usbaudio.h
> @@ -21,6 +21,15 @@ struct media_intf_devnode;
>  
>  #define MAX_CARD_INTERFACES	16
>  
> +/*
> + * Structure holding assosiation between Audio Control Interface
> + * and given Streaming or Midi Interface.
> + */
> +struct snd_intf_to_ctrl {
> +	u8 interface;
> +	struct usb_host_interface *ctrl_intf;
> +};
> +
>  struct snd_usb_audio {
>  	int index;
>  	struct usb_device *dev;
> @@ -63,6 +72,9 @@ struct snd_usb_audio {
>  	struct usb_host_interface *ctrl_intf;	/* the audio control interface */
>  	struct media_device *media_dev;
>  	struct media_intf_devnode *ctl_intf_media_devnode;
> +
> +	unsigned int num_intf_to_ctrl;
> +	struct snd_intf_to_ctrl intf_to_ctrl[MAX_CARD_INTERFACES];
>  };
>  
>  #define USB_AUDIO_IFACE_UNUSED	((void *)-1L)
> -- 
> 2.34.1
> 




[Index of Archives]     [Pulseaudio]     [Linux Audio Users]     [ALSA Devel]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux