soundwire: intel: fix wrong register name in intel_shim_wake
authorLibin Yang <libin.yang@intel.com>
Wed, 26 Jan 2022 01:14:51 +0000 (09:14 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 8 Apr 2022 12:23:44 +0000 (14:23 +0200)
[ Upstream commit 3957db3ae3dae6f8b8168791f154567fe49e1fd7 ]

When clearing the sdw wakests status, we should use SDW_SHIM_WAKESTS.

Fixes: 4a17c441c7cb ("soundwire: intel: revisit SHIM programming sequences.")
Signed-off-by: Libin Yang <libin.yang@intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20220126011451.27853-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/soundwire/intel.c

index 78037ff..f72d366 100644 (file)
@@ -448,8 +448,8 @@ static void intel_shim_wake(struct sdw_intel *sdw, bool wake_enable)
 
                /* Clear wake status */
                wake_sts = intel_readw(shim, SDW_SHIM_WAKESTS);
-               wake_sts |= (SDW_SHIM_WAKEEN_ENABLE << link_id);
-               intel_writew(shim, SDW_SHIM_WAKESTS_STATUS, wake_sts);
+               wake_sts |= (SDW_SHIM_WAKESTS_STATUS << link_id);
+               intel_writew(shim, SDW_SHIM_WAKESTS, wake_sts);
        }
        mutex_unlock(sdw->link_res->shim_lock);
 }