Skip to content

Commit

Permalink
ALSA: firewire-lib: remove WARN_ON() at destruction of AMDTP domain
Browse files Browse the repository at this point in the history
The destructor of AMDTP domain has WARN_ON() for the list of associated
AMDTP stream. Although this reminds a case that developers forget to
program consumer drivers to stop AMDTP domain, it hits when AMDTP domain
is not initialized yet. This occurs when initialization of sound card
fails as well and it's superfluous.

This commit removes the WARN_ON. Although the API to AMDTP domain does
nothing, it's left for future usage.

Fixes: 3ec3d7a ("ALSA: firewire-lib: add AMDTP domain structure to handle several isoc contexts")
Signed-off-by: Takashi Sakamoto <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Takashi Iwai <[email protected]>
  • Loading branch information
takaswie authored and tiwai committed Sep 7, 2019
1 parent e6e2fe2 commit 8d0d5c3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion sound/firewire/amdtp-stream.c
Original file line number Diff line number Diff line change
Expand Up @@ -1153,7 +1153,8 @@ EXPORT_SYMBOL_GPL(amdtp_domain_init);
*/
void amdtp_domain_destroy(struct amdtp_domain *d)
{
WARN_ON(!list_empty(&d->streams));
// At present nothing to do.
return;
}
EXPORT_SYMBOL_GPL(amdtp_domain_destroy);

Expand Down

0 comments on commit 8d0d5c3

Please sign in to comment.