thunderbolt: Fix bit masking in tb_dp_port_set_hops()
commit 2cdde91c14ec358087f43287513946d493aef940 upstream.
The tb_dp_port_set_hops() function was incorrectly clearing
ADP_DP_CS_1_AUX_RX_HOPID_MASK twice. According to the function's
purpose, it should clear both TX and RX AUX HopID fields. Replace the
first instance with ADP_DP_CS_1_AUX_TX_HOPID_MASK to ensure proper
configuration of both AUX directions.
Fixes: 98176380cb
("thunderbolt: Convert DP adapter register names to follow the USB4 spec")
Cc: stable@vger.kernel.org
Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
dc52aff534
commit
0cd051cb58
@@ -1465,7 +1465,7 @@ int tb_dp_port_set_hops(struct tb_port *port, unsigned int video,
|
||||
return ret;
|
||||
|
||||
data[0] &= ~ADP_DP_CS_0_VIDEO_HOPID_MASK;
|
||||
data[1] &= ~ADP_DP_CS_1_AUX_RX_HOPID_MASK;
|
||||
data[1] &= ~ADP_DP_CS_1_AUX_TX_HOPID_MASK;
|
||||
data[1] &= ~ADP_DP_CS_1_AUX_RX_HOPID_MASK;
|
||||
|
||||
data[0] |= (video << ADP_DP_CS_0_VIDEO_HOPID_SHIFT) &
|
||||
|
Reference in New Issue
Block a user