Commit f89830db authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman

thunderbolt: Fix reset response_type

commit 02729d17 upstream.

There is a mistake here where we accidentally use sizeof(TB_CFG_PKG_RESET)
instead of just TB_CFG_PKG_RESET.  The size of an int is 4 so it's the
same as TB_CFG_PKG_NOTIFY_ACK.

Fixes: d7f781bf ("thunderbolt: Rework control channel to be more reliable")
Reported-by: default avatarColin King <colin.king@canonical.com>
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 9808d1a5
...@@ -804,7 +804,7 @@ struct tb_cfg_result tb_cfg_reset(struct tb_ctl *ctl, u64 route, ...@@ -804,7 +804,7 @@ struct tb_cfg_result tb_cfg_reset(struct tb_ctl *ctl, u64 route,
req->request_type = TB_CFG_PKG_RESET; req->request_type = TB_CFG_PKG_RESET;
req->response = &reply; req->response = &reply;
req->response_size = sizeof(reply); req->response_size = sizeof(reply);
req->response_type = sizeof(TB_CFG_PKG_RESET); req->response_type = TB_CFG_PKG_RESET;
res = tb_cfg_request_sync(ctl, req, timeout_msec); res = tb_cfg_request_sync(ctl, req, timeout_msec);
......
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