soundwire: intel: fix potential race condition during power down
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Wed, 18 Aug 2021 02:49:52 +0000 (10:49 +0800)
committerVinod Koul <vkoul@kernel.org>
Mon, 23 Aug 2021 12:10:33 +0000 (17:40 +0530)
commitea6942dad4b2a7e1735aa0f10f3d0b04b847750f
treef6c41ea740884ada64ea62aaf39c4ab9562e3094
parentdd81e7c3f0bb4fbe48b770b389ab6ccb95058dab
soundwire: intel: fix potential race condition during power down

The power down sequence sets the link_up flag as false outside of the
mutex_lock. This is potentially unsafe.

In additional the flow in that sequence can be improved by first
testing if the link was powered, setting the link_up flag as false and
proceeding with the power down. In case the CPA bits cannot be
cleared, we only flag an error since we cannot deal with interrupts
any longer.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20210818024954.16873-2-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/soundwire/intel.c