ALSA: firewire: Replace tasklet with work
authorTakashi Iwai <tiwai@suse.de>
Wed, 9 Sep 2020 16:36:59 +0000 (18:36 +0200)
committerTakashi Iwai <tiwai@suse.de>
Fri, 11 Sep 2020 16:22:54 +0000 (18:22 +0200)
commit2b3d2987d800cc6dd8f6459971bc332354bd6a31
tree2e0baf9559c2654c40581b634533d60bc4a94e02
parent5554743d88483934cabff45943913be0830e66bc
ALSA: firewire: Replace tasklet with work

The tasklet is an old API that should be deprecated, usually can be
converted to another decent API.  In FireWire driver, a tasklet is
still used for offloading the AMDTP PCM stream handling.  It can be
achieved gracefully with a work queued, too.

This patch replaces the tasklet usage in firewire-lib driver with a
simple work.  The conversion is fairly straightforward but for the
in_interrupt() checks that are replaced with the check using the
current_work().

Note that in_interrupt() in amdtp_packet tracepoint is still kept as
is.  This is the place that is probed by both softirq of 1394 OHCI and
a user task of a PCM application, and the work handling is already
filtered in amdtp_domain_stream_pcm_pointer().

Tested-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Acked-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Link: https://lore.kernel.org/r/20200909163659.21708-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/amdtp-stream.c
sound/firewire/amdtp-stream.h