[PATCH v2 2/4] ALSA: firewire-lib: obsolete callbacked member

Takashi Sakamoto o-takashi at sakamocchi.jp
Mon May 24 05:13:44 CEST 2021


The member of callbacked in AMDTP stream structure is not used anymore.
Instead, ready_processing member is used to wake up yielding task of user
process.

Signed-off-by: Takashi Sakamoto <o-takashi at sakamocchi.jp>
---
 sound/firewire/amdtp-stream.c | 9 ++-------
 sound/firewire/amdtp-stream.h | 3 +--
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/sound/firewire/amdtp-stream.c b/sound/firewire/amdtp-stream.c
index 84608b856322..68ffbc33f692 100644
--- a/sound/firewire/amdtp-stream.c
+++ b/sound/firewire/amdtp-stream.c
@@ -110,7 +110,6 @@ int amdtp_stream_init(struct amdtp_stream *s, struct fw_unit *unit,
 	s->packet_index = 0;
 
 	init_waitqueue_head(&s->ready_wait);
-	s->callbacked = false;
 
 	s->fmt = fmt;
 	s->process_ctx_payloads = process_ctx_payloads;
@@ -1365,7 +1364,8 @@ static void irq_target_callback_skip(struct fw_iso_context *context, u32 tstamp,
 	d->processing_cycle.rx_start = cycle;
 }
 
-// this is executed one time.
+// This is executed one time. For in-stream, first packet has come. For out-stream, prepared to
+// transmit first packet.
 static void amdtp_stream_first_callback(struct fw_iso_context *context,
 					u32 tstamp, size_t header_length,
 					void *header, void *private_data)
@@ -1373,10 +1373,6 @@ static void amdtp_stream_first_callback(struct fw_iso_context *context,
 	struct amdtp_stream *s = private_data;
 	struct amdtp_domain *d = s->domain;
 
-	// For in-stream, first packet has come.
-	// For out-stream, prepared to transmit first packet
-	s->callbacked = true;
-
 	if (s->direction == AMDTP_IN_STREAM) {
 		context->callback.sc = drop_tx_packets_initially;
 	} else {
@@ -1536,7 +1532,6 @@ static int amdtp_stream_start(struct amdtp_stream *s, int channel, int speed,
 	if ((s->flags & CIP_EMPTY_WITH_TAG0) || (s->flags & CIP_NO_HEADER))
 		tag |= FW_ISO_CONTEXT_MATCH_TAG0;
 
-	s->callbacked = false;
 	s->ready_processing = false;
 	err = fw_iso_context_start(s->context, -1, 0, tag);
 	if (err < 0)
diff --git a/sound/firewire/amdtp-stream.h b/sound/firewire/amdtp-stream.h
index d3ba2e1c1522..34294776f9e8 100644
--- a/sound/firewire/amdtp-stream.h
+++ b/sound/firewire/amdtp-stream.h
@@ -183,8 +183,7 @@ struct amdtp_stream {
 
 	// To start processing content of packets at the same cycle in several contexts for
 	// each direction.
-	bool callbacked:1;
-	bool ready_processing:1;
+	bool ready_processing;
 	wait_queue_head_t ready_wait;
 	unsigned int next_cycle;
 
-- 
2.27.0



More information about the Alsa-devel mailing list