On Wed, Dec 18, 2024 at 03:56:48PM +0530, Prashanth K wrote: > Enabling U1 and U2 power-saving states can lead to stability and > performance issues, particularly for latency-sensitive or high- > throughput applications. These low-power link states are intended > to reduce power consumption by allowing the device to enter partial > low-power modes during idle periods. However, they can sometimes > result in unexpected behavior. Over the years, some of the issues > seen are as follows: > [..] > > This series was earlier started by Krishna Kurapati where he disabled > U1/U2 on some SM targets. I'm extending this to more devices including > Auto, Compute and IOT platforms. On a side note, this quirk has been > already included on some mobile targets like SM8550/8650. Why are you resending previous patches rather than adding another series on top of it? > > Changes in v2: > - Removed the wrongly added quirks from tcsr_mutex node. > - Link to v2: https://lore.kernel.org/all/20241213095237.1409174-1-quic_prashk@xxxxxxxxxxx/ What was changed in v3? > > Link to RFC: > https://lore.kernel.org/all/20241107073650.13473-1-quic_kriskura@xxxxxxxxxxx/#Z31arch:arm64:boot:dts:qcom:sm8250.dtsi > -- With best wishes Dmitry
