diff options
author | Takashi Iwai <tiwai@suse.de> | 2022-10-09 13:42:12 +0300 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2022-10-09 13:43:26 +0300 |
commit | 1045f5f1ff0751423aeb65648e5e1abd7a7a8672 (patch) | |
tree | 595dd8f43057933559def9ba5459c024afff35b3 /sound/usb/card.h | |
parent | 9355b60e401d825590d37f04ea873c58efe9b7bf (diff) | |
download | linux-1045f5f1ff0751423aeb65648e5e1abd7a7a8672.tar.xz |
ALSA: usb-audio: Avoid superfluous endpoint setup
After splitting to snd_usb_endpoint_set_params() and *_prepare(), the
skip of each function should be checked with different flags, while we
still use ep->need_setup as the single one. Introduce
ep->need_prepare for indicating the need of prepare, and also add the
missing check of ep->need_setup at the set_params.
Fixes: 2be79d586454 ("ALSA: usb-audio: Split endpoint setups for hw_params and prepare (take#2)")
Link: https://lore.kernel.org/r/20221009104212.18877-5-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/usb/card.h')
-rw-r--r-- | sound/usb/card.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/usb/card.h b/sound/usb/card.h index ca75f2206170..40061550105a 100644 --- a/sound/usb/card.h +++ b/sound/usb/card.h @@ -129,7 +129,8 @@ struct snd_usb_endpoint { in a stream */ bool implicit_fb_sync; /* syncs with implicit feedback */ bool lowlatency_playback; /* low-latency playback mode */ - bool need_setup; /* (re-)need for configure? */ + bool need_setup; /* (re-)need for hw_params? */ + bool need_prepare; /* (re-)need for prepare? */ /* for hw constraints */ const struct audioformat *cur_audiofmt; |