scsi: ufs: Fix system suspend status

Stanley Chu · martinkpetersen · commit ce9e7bce4352 · 2019-01-11T22:16:37.000-05:00
hba-&gt;is_sys_suspended is set after successful system suspend but
not clear after successful system resume.

According to current behavior, hba-&gt;is_sys_suspended will not be set if
host is runtime-suspended but not system-suspended. Thus we shall aligh the
same policy: clear this flag even if host remains runtime-suspended after
ufshcd_system_resume is successfully returned.

Simply fix this flag to correct host status logs.

Signed-off-by: Stanley Chu &lt;stanley.chu@mediatek.com&gt;
Reviewed-by: Avri Altman &lt;avri.altman@wdc.com&gt;
Signed-off-by: Martin K. Petersen &lt;martin.petersen@oracle.com&gt;
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
@@ -8001,6 +8001,8 @@ int ufshcd_system_resume(struct ufs_hba *hba)
 	trace_ufshcd_system_resume(dev_name(hba->dev), ret,
 		ktime_to_us(ktime_sub(ktime_get(), start)),
 		hba->curr_dev_pwr_mode, hba->uic_link_state);
+	if (!ret)
+		hba->is_sys_suspended = false;
 	return ret;
 }
 EXPORT_SYMBOL(ufshcd_system_resume);

Original file line number	Diff line number	Diff line change
`@@ -8001,6 +8001,8 @@ int ufshcd_system_resume(struct ufs_hba *hba)`
`8001`	`8001`	`trace_ufshcd_system_resume(dev_name(hba->dev), ret,`
`8002`	`8002`	`ktime_to_us(ktime_sub(ktime_get(), start)),`
`8003`	`8003`	`hba->curr_dev_pwr_mode, hba->uic_link_state);`
	`8004`	`+ if (!ret)`
	`8005`	`+ hba->is_sys_suspended = false;`
`8004`	`8006`	`return ret;`
`8005`	`8007`	`}`
`8006`	`8008`	`EXPORT_SYMBOL(ufshcd_system_resume);`