Commit 8a96e7be authored by Takashi Sakamoto's avatar Takashi Sakamoto

firewire: ohci: use static inline functions to serialize data of IT DMA

THis commit replaces current implementation with the helper functions added
in the former commit.

Link: https://lore.kernel.org/r/20240802003606.109402-5-o-takashi@sakamocchi.jpSigned-off-by: default avatarTakashi Sakamoto <o-takashi@sakamocchi.jp>
parent 1ce2a92b
...@@ -162,13 +162,6 @@ struct context { ...@@ -162,13 +162,6 @@ struct context {
struct tasklet_struct tasklet; struct tasklet_struct tasklet;
}; };
#define IT_HEADER_SY(v) ((v) << 0)
#define IT_HEADER_TCODE(v) ((v) << 4)
#define IT_HEADER_CHANNEL(v) ((v) << 8)
#define IT_HEADER_TAG(v) ((v) << 14)
#define IT_HEADER_SPEED(v) ((v) << 16)
#define IT_HEADER_DATA_LENGTH(v) ((v) << 16)
struct iso_context { struct iso_context {
struct fw_iso_context base; struct fw_iso_context base;
struct context context; struct context context;
...@@ -1457,9 +1450,14 @@ static int at_context_queue_packet(struct context *ctx, ...@@ -1457,9 +1450,14 @@ static int at_context_queue_packet(struct context *ctx,
break; break;
case TCODE_STREAM_DATA: case TCODE_STREAM_DATA:
header[0] = cpu_to_le32((packet->header[0] & 0xffff) | ohci1394_it_data_set_speed(header, packet->speed);
(packet->speed << 16)); ohci1394_it_data_set_tag(header, isoc_header_get_tag(packet->header[0]));
header[1] = cpu_to_le32(packet->header[0] & 0xffff0000); ohci1394_it_data_set_channel(header, isoc_header_get_channel(packet->header[0]));
ohci1394_it_data_set_tcode(header, TCODE_STREAM_DATA);
ohci1394_it_data_set_sync(header, isoc_header_get_sy(packet->header[0]));
ohci1394_it_data_set_data_length(header, isoc_header_get_data_length(packet->header[0]));
d[0].req_count = cpu_to_le16(8); d[0].req_count = cpu_to_le16(8);
break; break;
...@@ -3403,14 +3401,14 @@ static int queue_iso_transmit(struct iso_context *ctx, ...@@ -3403,14 +3401,14 @@ static int queue_iso_transmit(struct iso_context *ctx,
d[0].branch_address = cpu_to_le32(d_bus | z); d[0].branch_address = cpu_to_le32(d_bus | z);
header = (__le32 *) &d[1]; header = (__le32 *) &d[1];
header[0] = cpu_to_le32(IT_HEADER_SY(p->sy) |
IT_HEADER_TAG(p->tag) | ohci1394_it_data_set_speed(header, ctx->base.speed);
IT_HEADER_TCODE(TCODE_STREAM_DATA) | ohci1394_it_data_set_tag(header, p->tag);
IT_HEADER_CHANNEL(ctx->base.channel) | ohci1394_it_data_set_channel(header, ctx->base.channel);
IT_HEADER_SPEED(ctx->base.speed)); ohci1394_it_data_set_tcode(header, TCODE_STREAM_DATA);
header[1] = ohci1394_it_data_set_sync(header, p->sy);
cpu_to_le32(IT_HEADER_DATA_LENGTH(p->header_length +
p->payload_length)); ohci1394_it_data_set_data_length(header, p->header_length + p->payload_length);
} }
if (p->header_length > 0) { if (p->header_length > 0) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment