diff options
| -rw-r--r-- | sound/usb/endpoint.c | 17 | ||||
| -rw-r--r-- | sound/usb/endpoint.h | 2 | ||||
| -rw-r--r-- | sound/usb/pcm.c | 10 |
3 files changed, 13 insertions, 16 deletions
diff --git a/sound/usb/endpoint.c b/sound/usb/endpoint.c index 15d1d5c63c3c..2f0ea70a998c 100644 --- a/sound/usb/endpoint.c +++ b/sound/usb/endpoint.c | |||
| @@ -534,6 +534,11 @@ static int wait_clear_urbs(struct snd_usb_endpoint *ep) | |||
| 534 | alive, ep->ep_num); | 534 | alive, ep->ep_num); |
| 535 | clear_bit(EP_FLAG_STOPPING, &ep->flags); | 535 | clear_bit(EP_FLAG_STOPPING, &ep->flags); |
| 536 | 536 | ||
| 537 | ep->data_subs = NULL; | ||
| 538 | ep->sync_slave = NULL; | ||
| 539 | ep->retire_data_urb = NULL; | ||
| 540 | ep->prepare_data_urb = NULL; | ||
| 541 | |||
| 537 | return 0; | 542 | return 0; |
| 538 | } | 543 | } |
| 539 | 544 | ||
| @@ -912,9 +917,7 @@ int snd_usb_endpoint_set_params(struct snd_usb_endpoint *ep, | |||
| 912 | /** | 917 | /** |
| 913 | * snd_usb_endpoint_start: start an snd_usb_endpoint | 918 | * snd_usb_endpoint_start: start an snd_usb_endpoint |
| 914 | * | 919 | * |
| 915 | * @ep: the endpoint to start | 920 | * @ep: the endpoint to start |
| 916 | * @can_sleep: flag indicating whether the operation is executed in | ||
| 917 | * non-atomic context | ||
| 918 | * | 921 | * |
| 919 | * A call to this function will increment the use count of the endpoint. | 922 | * A call to this function will increment the use count of the endpoint. |
| 920 | * In case it is not already running, the URBs for this endpoint will be | 923 | * In case it is not already running, the URBs for this endpoint will be |
| @@ -924,7 +927,7 @@ int snd_usb_endpoint_set_params(struct snd_usb_endpoint *ep, | |||
| 924 | * | 927 | * |
| 925 | * Returns an error if the URB submission failed, 0 in all other cases. | 928 | * Returns an error if the URB submission failed, 0 in all other cases. |
| 926 | */ | 929 | */ |
| 927 | int snd_usb_endpoint_start(struct snd_usb_endpoint *ep, bool can_sleep) | 930 | int snd_usb_endpoint_start(struct snd_usb_endpoint *ep) |
| 928 | { | 931 | { |
| 929 | int err; | 932 | int err; |
| 930 | unsigned int i; | 933 | unsigned int i; |
| @@ -938,8 +941,6 @@ int snd_usb_endpoint_start(struct snd_usb_endpoint *ep, bool can_sleep) | |||
| 938 | 941 | ||
| 939 | /* just to be sure */ | 942 | /* just to be sure */ |
| 940 | deactivate_urbs(ep, false); | 943 | deactivate_urbs(ep, false); |
| 941 | if (can_sleep) | ||
| 942 | wait_clear_urbs(ep); | ||
| 943 | 944 | ||
| 944 | ep->active_mask = 0; | 945 | ep->active_mask = 0; |
| 945 | ep->unlink_mask = 0; | 946 | ep->unlink_mask = 0; |
| @@ -1020,10 +1021,6 @@ void snd_usb_endpoint_stop(struct snd_usb_endpoint *ep) | |||
| 1020 | 1021 | ||
| 1021 | if (--ep->use_count == 0) { | 1022 | if (--ep->use_count == 0) { |
| 1022 | deactivate_urbs(ep, false); | 1023 | deactivate_urbs(ep, false); |
| 1023 | ep->data_subs = NULL; | ||
| 1024 | ep->sync_slave = NULL; | ||
| 1025 | ep->retire_data_urb = NULL; | ||
| 1026 | ep->prepare_data_urb = NULL; | ||
| 1027 | set_bit(EP_FLAG_STOPPING, &ep->flags); | 1024 | set_bit(EP_FLAG_STOPPING, &ep->flags); |
| 1028 | } | 1025 | } |
| 1029 | } | 1026 | } |
diff --git a/sound/usb/endpoint.h b/sound/usb/endpoint.h index 6428392d8f62..584f295d7c77 100644 --- a/sound/usb/endpoint.h +++ b/sound/usb/endpoint.h | |||
| @@ -18,7 +18,7 @@ int snd_usb_endpoint_set_params(struct snd_usb_endpoint *ep, | |||
| 18 | struct audioformat *fmt, | 18 | struct audioformat *fmt, |
| 19 | struct snd_usb_endpoint *sync_ep); | 19 | struct snd_usb_endpoint *sync_ep); |
| 20 | 20 | ||
| 21 | int snd_usb_endpoint_start(struct snd_usb_endpoint *ep, bool can_sleep); | 21 | int snd_usb_endpoint_start(struct snd_usb_endpoint *ep); |
| 22 | void snd_usb_endpoint_stop(struct snd_usb_endpoint *ep); | 22 | void snd_usb_endpoint_stop(struct snd_usb_endpoint *ep); |
| 23 | void snd_usb_endpoint_sync_pending_stop(struct snd_usb_endpoint *ep); | 23 | void snd_usb_endpoint_sync_pending_stop(struct snd_usb_endpoint *ep); |
| 24 | int snd_usb_endpoint_activate(struct snd_usb_endpoint *ep); | 24 | int snd_usb_endpoint_activate(struct snd_usb_endpoint *ep); |
diff --git a/sound/usb/pcm.c b/sound/usb/pcm.c index 34c6d4f2c0b6..9aa5b1855481 100644 --- a/sound/usb/pcm.c +++ b/sound/usb/pcm.c | |||
| @@ -218,7 +218,7 @@ int snd_usb_init_pitch(struct snd_usb_audio *chip, int iface, | |||
| 218 | } | 218 | } |
| 219 | } | 219 | } |
| 220 | 220 | ||
| 221 | static int start_endpoints(struct snd_usb_substream *subs, bool can_sleep) | 221 | static int start_endpoints(struct snd_usb_substream *subs) |
| 222 | { | 222 | { |
| 223 | int err; | 223 | int err; |
| 224 | 224 | ||
| @@ -231,7 +231,7 @@ static int start_endpoints(struct snd_usb_substream *subs, bool can_sleep) | |||
| 231 | dev_dbg(&subs->dev->dev, "Starting data EP @%p\n", ep); | 231 | dev_dbg(&subs->dev->dev, "Starting data EP @%p\n", ep); |
| 232 | 232 | ||
| 233 | ep->data_subs = subs; | 233 | ep->data_subs = subs; |
| 234 | err = snd_usb_endpoint_start(ep, can_sleep); | 234 | err = snd_usb_endpoint_start(ep); |
| 235 | if (err < 0) { | 235 | if (err < 0) { |
| 236 | clear_bit(SUBSTREAM_FLAG_DATA_EP_STARTED, &subs->flags); | 236 | clear_bit(SUBSTREAM_FLAG_DATA_EP_STARTED, &subs->flags); |
| 237 | return err; | 237 | return err; |
| @@ -260,7 +260,7 @@ static int start_endpoints(struct snd_usb_substream *subs, bool can_sleep) | |||
| 260 | dev_dbg(&subs->dev->dev, "Starting sync EP @%p\n", ep); | 260 | dev_dbg(&subs->dev->dev, "Starting sync EP @%p\n", ep); |
| 261 | 261 | ||
| 262 | ep->sync_slave = subs->data_endpoint; | 262 | ep->sync_slave = subs->data_endpoint; |
| 263 | err = snd_usb_endpoint_start(ep, can_sleep); | 263 | err = snd_usb_endpoint_start(ep); |
| 264 | if (err < 0) { | 264 | if (err < 0) { |
| 265 | clear_bit(SUBSTREAM_FLAG_SYNC_EP_STARTED, &subs->flags); | 265 | clear_bit(SUBSTREAM_FLAG_SYNC_EP_STARTED, &subs->flags); |
| 266 | return err; | 266 | return err; |
| @@ -850,7 +850,7 @@ static int snd_usb_pcm_prepare(struct snd_pcm_substream *substream) | |||
| 850 | /* for playback, submit the URBs now; otherwise, the first hwptr_done | 850 | /* for playback, submit the URBs now; otherwise, the first hwptr_done |
| 851 | * updates for all URBs would happen at the same time when starting */ | 851 | * updates for all URBs would happen at the same time when starting */ |
| 852 | if (subs->direction == SNDRV_PCM_STREAM_PLAYBACK) | 852 | if (subs->direction == SNDRV_PCM_STREAM_PLAYBACK) |
| 853 | ret = start_endpoints(subs, true); | 853 | ret = start_endpoints(subs); |
| 854 | 854 | ||
| 855 | unlock: | 855 | unlock: |
| 856 | snd_usb_unlock_shutdown(subs->stream->chip); | 856 | snd_usb_unlock_shutdown(subs->stream->chip); |
| @@ -1666,7 +1666,7 @@ static int snd_usb_substream_capture_trigger(struct snd_pcm_substream *substream | |||
| 1666 | 1666 | ||
| 1667 | switch (cmd) { | 1667 | switch (cmd) { |
| 1668 | case SNDRV_PCM_TRIGGER_START: | 1668 | case SNDRV_PCM_TRIGGER_START: |
| 1669 | err = start_endpoints(subs, false); | 1669 | err = start_endpoints(subs); |
| 1670 | if (err < 0) | 1670 | if (err < 0) |
| 1671 | return err; | 1671 | return err; |
| 1672 | 1672 | ||
