]> www.infradead.org Git - users/hch/dma-mapping.git/commit
soundwire: intel_bus_common: enable interrupts before exiting reset
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Mon, 5 Aug 2024 11:50:03 +0000 (19:50 +0800)
committerVinod Koul <vkoul@kernel.org>
Sun, 18 Aug 2024 06:44:53 +0000 (12:14 +0530)
commit5aedb8d8336b0a0421b58ca27d1b572aa6695b5b
treecdb3485b48adef6a5bee4b0de2f660f024e5aa03
parentf8c35d61ba01afa76846905c67862cdace7f66b0
soundwire: intel_bus_common: enable interrupts before exiting reset

The existing code enables the Cadence IP interrupts after the bus
reset sequence. The problem with this sequence is that it might be
pre-empted, giving SoundWire devices time to sync and report as
ATTACHED before the interrupts are enabled. In that case, the Cadence
IP will not detect a state change and will not throw an interrupt to
proceed with the enumeration of a Device0.

In our overnight stress tests, we observed that a slight
sub-millisecond delay in enabling interrupts after the reset was
correlated with detection failures. This problem is more prevalent on
the LunarLake silicon, likely due to SOC integration changes, but it
was observed on earlier generations as well.

This patch reverts the sequence, with the interrupts enabled before
performing the bus reset. This removes the race condition and makes
sure the Cadence IP is able to detect the presence of a Device0 in all
cases.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20240805115003.88035-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/soundwire/intel_bus_common.c