On Mon, Dec 05, 2022 at 01:33:06PM -0600, Pierre-Louis Bossart wrote:
On 12/5/22 12:33, Greg KH wrote:
On Mon, Dec 05, 2022 at 11:06:00AM -0600, Pierre-Louis Bossart wrote:
From: Sjoerd Simons sjoerd@collabora.com
commit 13c30a755847c7e804e1bf755e66e3ff7b7f9367 upstream
The bus->clk_stop_timeout member is only initialized to a non-zero value during the codec driver probe. This can lead to corner cases where this value remains pegged at zero when the bus suspends, which results in an endless loop in sdw_bus_wait_for_clk_prep_deprep().
Corner cases include configurations with no codecs described in the firmware, or delays in probing codec drivers.
Initializing the default timeout to the smallest non-zero value avoid this problem and allows for the existing logic to be preserved: the bus->clk_stop_timeout is set as the maximum required by all codecs connected on the bus.
Fixes: 1f2dcf3a154ac ("soundwire: intel: set dev_num_ida_min")
This commit is is only in 6.1-rc1, so why does it need to go to any older kernels? Is this tag not correct?
I don't recall why this tag was selected, it's clearly not related functionality-wise. I vaguely recall a discussion with Bard Liao on this... And yes sure enough here it is [1], it was to indicate a conflict but that was confusing in hindsight.
At any rate, this one-line change is really needed, some distributions such as Arch back-ported this change but most did not, and users don't have a working setup.
Ok, now queued up, thanks.
greg k-h