OSDN Git Service

ALSA: firewire-lib: split helper function to generate CIP header
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Thu, 23 May 2019 15:14:37 +0000 (00:14 +0900)
committerTakashi Iwai <tiwai@suse.de>
Fri, 24 May 2019 07:05:15 +0000 (09:05 +0200)
This is minor code refactoring to split a function to generate CIP
header.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/amdtp-stream.c

index 2d9c764..2598566 100644 (file)
@@ -477,6 +477,19 @@ static inline int queue_in_packet(struct amdtp_stream *s)
        return queue_packet(s, s->ctx_data.tx.max_ctx_payload_length);
 }
 
+static void generate_cip_header(struct amdtp_stream *s, __be32 cip_header[2],
+                               unsigned int syt)
+{
+       cip_header[0] = cpu_to_be32(READ_ONCE(s->source_node_id_field) |
+                               (s->data_block_quadlets << CIP_DBS_SHIFT) |
+                               ((s->sph << CIP_SPH_SHIFT) & CIP_SPH_MASK) |
+                               s->data_block_counter);
+       cip_header[1] = cpu_to_be32(CIP_EOH |
+                       ((s->fmt << CIP_FMT_SHIFT) & CIP_FMT_MASK) |
+                       ((s->ctx_data.rx.fdf << CIP_FDF_SHIFT) & CIP_FDF_MASK) |
+                       (syt & CIP_SYT_MASK));
+}
+
 static int handle_out_packet(struct amdtp_stream *s, unsigned int cycle,
                             const __be32 *ctx_header, __be32 *buffer,
                             unsigned int index)
@@ -495,14 +508,7 @@ static int handle_out_packet(struct amdtp_stream *s, unsigned int cycle,
                s->data_block_counter =
                                (s->data_block_counter + data_blocks) & 0xff;
 
-       buffer[0] = cpu_to_be32(READ_ONCE(s->source_node_id_field) |
-                               (s->data_block_quadlets << CIP_DBS_SHIFT) |
-                               ((s->sph << CIP_SPH_SHIFT) & CIP_SPH_MASK) |
-                               s->data_block_counter);
-       buffer[1] = cpu_to_be32(CIP_EOH |
-                       ((s->fmt << CIP_FMT_SHIFT) & CIP_FMT_MASK) |
-                       ((s->ctx_data.rx.fdf << CIP_FDF_SHIFT) & CIP_FDF_MASK) |
-                       (syt & CIP_SYT_MASK));
+       generate_cip_header(s, buffer, syt);
 
        if (!(s->flags & CIP_DBC_IS_END_EVENT))
                s->data_block_counter =