Hi Gil, On Mon, Dec 27, 2021 at 06:13:44PM +0200, Gil Fine wrote: > Hi Mika, > About latest patches series adding CL0s in thunderbolt/next.git > 1. > Attached fix for CL0s enabled check - w/o it, the CL0s enabling fails Thanks, I folded this to patch "thunderbolt: Enable CL0s for Intel Titan Ridge". > 2. > Maybe it is better not to mention "Intel limitation..." about the first-hop > router since I am not sure this is correct. Attached patch w/ my proposal. Well it is targeted only for Intel Titan Ridge so I think we can keep it for documentation purposes :) > 3. > Don't we prefer to stay with the name: "tb_switch_titan_ridge_pcie_l1_enable" ? > so that it is clear that the function is Titan Ridge specific? > If so, sending patch for this It works with any router (as it specifically checks for Titan Ridge before going further), I think we can keep the name more generic too.
