Commit e5876559 authored by Mika Westerberg's avatar Mika Westerberg

thunderbolt: Use dedicated flow control for DMA tunnels

The USB4 inter-domain service spec recommends using dedicated flow
control scheme so update the driver accordingly.
Signed-off-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
parent 46b494f2
...@@ -794,24 +794,14 @@ static u32 tb_dma_credits(struct tb_port *nhi) ...@@ -794,24 +794,14 @@ static u32 tb_dma_credits(struct tb_port *nhi)
return min(max_credits, 13U); return min(max_credits, 13U);
} }
static int tb_dma_activate(struct tb_tunnel *tunnel, bool active) static void tb_dma_init_path(struct tb_path *path, unsigned int efc, u32 credits)
{
struct tb_port *nhi = tunnel->src_port;
u32 credits;
credits = active ? tb_dma_credits(nhi) : 0;
return tb_port_set_initial_credits(nhi, credits);
}
static void tb_dma_init_path(struct tb_path *path, unsigned int isb,
unsigned int efc, u32 credits)
{ {
int i; int i;
path->egress_fc_enable = efc; path->egress_fc_enable = efc;
path->ingress_fc_enable = TB_PATH_ALL; path->ingress_fc_enable = TB_PATH_ALL;
path->egress_shared_buffer = TB_PATH_NONE; path->egress_shared_buffer = TB_PATH_NONE;
path->ingress_shared_buffer = isb; path->ingress_shared_buffer = TB_PATH_NONE;
path->priority = 5; path->priority = 5;
path->weight = 1; path->weight = 1;
path->clear_fc = true; path->clear_fc = true;
...@@ -856,7 +846,6 @@ struct tb_tunnel *tb_tunnel_alloc_dma(struct tb *tb, struct tb_port *nhi, ...@@ -856,7 +846,6 @@ struct tb_tunnel *tb_tunnel_alloc_dma(struct tb *tb, struct tb_port *nhi,
if (!tunnel) if (!tunnel)
return NULL; return NULL;
tunnel->activate = tb_dma_activate;
tunnel->src_port = nhi; tunnel->src_port = nhi;
tunnel->dst_port = dst; tunnel->dst_port = dst;
...@@ -869,8 +858,7 @@ struct tb_tunnel *tb_tunnel_alloc_dma(struct tb *tb, struct tb_port *nhi, ...@@ -869,8 +858,7 @@ struct tb_tunnel *tb_tunnel_alloc_dma(struct tb *tb, struct tb_port *nhi,
tb_tunnel_free(tunnel); tb_tunnel_free(tunnel);
return NULL; return NULL;
} }
tb_dma_init_path(path, TB_PATH_NONE, TB_PATH_SOURCE | TB_PATH_INTERNAL, tb_dma_init_path(path, TB_PATH_SOURCE | TB_PATH_INTERNAL, credits);
credits);
tunnel->paths[i++] = path; tunnel->paths[i++] = path;
} }
...@@ -881,7 +869,7 @@ struct tb_tunnel *tb_tunnel_alloc_dma(struct tb *tb, struct tb_port *nhi, ...@@ -881,7 +869,7 @@ struct tb_tunnel *tb_tunnel_alloc_dma(struct tb *tb, struct tb_port *nhi,
tb_tunnel_free(tunnel); tb_tunnel_free(tunnel);
return NULL; return NULL;
} }
tb_dma_init_path(path, TB_PATH_SOURCE, TB_PATH_ALL, credits); tb_dma_init_path(path, TB_PATH_ALL, credits);
tunnel->paths[i++] = path; tunnel->paths[i++] = path;
} }
......
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