On Tue, Jul 11, 2023 at 02:27:50AM -0500, Sanjay R Mehta wrote: > From: Sanjay R Mehta <sanju.mehta@xxxxxxx> > > Previously, on unplug events, the TMU mode was disabled first > followed by the Time Synchronization Handshake, irrespective of > whether the tb_switch_tmu_rate_write() API was successful or not. > > However, this caused a problem with Thunderbolt 3 (TBT3) > devices, as the TSPacketInterval bits were always enabled by default, > leading the host router to assume that the device router's TMU was > already enabled and preventing it from initiating the Time > Synchronization Handshake. As a result, TBT3 monitors experienced > display flickering from the second hot plug onwards. > > To address this issue, we have modified the code to only disable the > Time Synchronization Handshake during TMU disable if the > tb_switch_tmu_rate_write() function is successful. This ensures that > the TBT3 devices function correctly and eliminates the display > flickering issue. > > Signed-off-by: Sanath S <Sanath.S@xxxxxxx> > Signed-off-by: Sanjay R Mehta <sanju.mehta@xxxxxxx> If "Sanath S" authored this patch, then the From: line is incorrect. If they did not, then the ordering here is incorrect. Either way, this isn't correct at all, please fix. thanks, greg k-h