Commit b6322f77 authored Nov 25, 2020 by Ziqi Chen

scsi: ufs: Don't disable core_clk_unipro if the link is active



If we want to disable clocks but still keep the link active, both
ref_clk and core_clk_unipro should be skipped.

Commit 284fa080cb5f ("scsi: ufs: fix broken hibern8_on_idle sysfs
nodes") is the original change from msm-4.14. We pick it up to msm-5.4
due to upstream missed this change. To follow GKI policy on 5.4, we
simplifed it and kept parameters name to avoid breaking ABI.

Change-Id: I5f86775c150e495322d14ec57ebf76a710257931
Signed-off-by: Ziqi Chen <ziqichen@codeaurora.org>

parent e8b3c550

drivers/scsi/ufs/ufshcd.c

+10 −0

Original line number	Diff line number	Diff line
		@@ -8246,7 +8246,17 @@ static int __ufshcd_setup_clocks(struct ufs_hba *hba, bool on,

		list_for_each_entry(clki, head, list) {
		if (!IS_ERR_OR_NULL(clki->clk)) {
		#if defined(CONFIG_SCSI_UFSHCD_QTI)
		/*
		* To keep link active, ref_clk and core_clk_unipro
		* should be kept ON.
		*/
		if (skip_ref_clk &&
		(!strcmp(clki->name, "ref_clk") \|\|
		!strcmp(clki->name, "core_clk_unipro")))
		#else
		if (skip_ref_clk && !strcmp(clki->name, "ref_clk"))
		#endif
		continue;

		clk_state_changed = on ^ clki->enabled;