[PATCH v2 Cover Letter] thunderbolt: Disable Gen 4 Recovery on Asymmetric Transitions

[Mohammad Rahimi <rahimi.mhmmd@gmail.com>]

Dear Mika,

In this revision, I have decided not to define the following functions,
despite our initial agreement in v1:

int tb_enable_link_recovery(struct tb_switch *sw);
int tb_disable_link_recovery(struct tb_switch *sw);

This decision was based on the following considerations:

1. If a write operation fails, there is no clear way to determine which port
the failure occurred on, the upstream port of the switch or its link partner.

2. To restore the exact Link Recovery state after an asymmetric transition, I
would need to track two boolean parameters and pass them to
tb_disable_link_recovery(). This would expose port state logic to
tb_configure_asym(), ending up with the same design we already have.

Additionally, I have chosen to ignore the return value from
usb4_port_link_recovery_enable() in tb_configure_asym(). At that stage, we
cannot take meaningful action based on the return value. Moreover, re-enabling
is performed immediately after tb_switch_set_link_width() to avoid introducing
unnecessary conditional branches.

Please let me know if you have any concerns or suggestions.

Best regards,
Mohammad Rahimi