mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-02-17 19:30:11 -05:00
scsi: ufs: ufs-pci: Fix hibernate state transition for Intel MTL-like host controllers
UFSHCD core disables the UIC completion interrupt when issuing UIC
hibernation commands, and re-enables it afterwards if it was enabled to
start with, refer ufshcd_uic_pwr_ctrl(). For Intel MTL-like host
controllers, accessing the register to re-enable the interrupt disrupts
the state transition.
Use hibern8_notify variant operation to disable the interrupt during the
entire hibernation, thereby preventing the disruption.
Fixes: 4049f7acef ("scsi: ufs: ufs-pci: Add support for Intel MTL")
Cc: stable@vger.kernel.org
Signed-off-by: Archana Patni <archana.patni@intel.com>
Link: https://lore.kernel.org/r/20250723165856.145750-2-adrian.hunter@intel.com
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
committed by
Martin K. Petersen
parent
19272b37aa
commit
4428ddea83
@@ -216,6 +216,32 @@ static int ufs_intel_lkf_apply_dev_quirks(struct ufs_hba *hba)
|
||||
return ret;
|
||||
}
|
||||
|
||||
static void ufs_intel_ctrl_uic_compl(struct ufs_hba *hba, bool enable)
|
||||
{
|
||||
u32 set = ufshcd_readl(hba, REG_INTERRUPT_ENABLE);
|
||||
|
||||
if (enable)
|
||||
set |= UIC_COMMAND_COMPL;
|
||||
else
|
||||
set &= ~UIC_COMMAND_COMPL;
|
||||
ufshcd_writel(hba, set, REG_INTERRUPT_ENABLE);
|
||||
}
|
||||
|
||||
static void ufs_intel_mtl_h8_notify(struct ufs_hba *hba,
|
||||
enum uic_cmd_dme cmd,
|
||||
enum ufs_notify_change_status status)
|
||||
{
|
||||
/*
|
||||
* Disable UIC COMPL INTR to prevent access to UFSHCI after
|
||||
* checking HCS.UPMCRS
|
||||
*/
|
||||
if (status == PRE_CHANGE && cmd == UIC_CMD_DME_HIBER_ENTER)
|
||||
ufs_intel_ctrl_uic_compl(hba, false);
|
||||
|
||||
if (status == POST_CHANGE && cmd == UIC_CMD_DME_HIBER_EXIT)
|
||||
ufs_intel_ctrl_uic_compl(hba, true);
|
||||
}
|
||||
|
||||
#define INTEL_ACTIVELTR 0x804
|
||||
#define INTEL_IDLELTR 0x808
|
||||
|
||||
@@ -533,6 +559,7 @@ static struct ufs_hba_variant_ops ufs_intel_mtl_hba_vops = {
|
||||
.init = ufs_intel_mtl_init,
|
||||
.exit = ufs_intel_common_exit,
|
||||
.hce_enable_notify = ufs_intel_hce_enable_notify,
|
||||
.hibern8_notify = ufs_intel_mtl_h8_notify,
|
||||
.link_startup_notify = ufs_intel_link_startup_notify,
|
||||
.resume = ufs_intel_resume,
|
||||
.device_reset = ufs_intel_device_reset,
|
||||
|
||||
Reference in New Issue
Block a user