From: Yaniv Gardi <ygardi@codeaurora.org>
To: robherring2@gmail.com, James.Bottomley@HansenPartnership.com,
pebolle@tiscali.nl, hch@infradead.org
Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org,
linux-arm-msm@vger.kernel.org, santoshsy@gmail.com,
linux-scsi-owner@vger.kernel.org, subhashj@codeaurora.org,
ygardi@codeaurora.org, Vinayak Holikatti <vinholikatti@gmail.com>,
"James E.J. Bottomley" <JBottomley@odin.com>,
"Martin K. Petersen" <martin.petersen@oracle.com>
Subject: [v3 12/14] scsi: ufs: fix leakage during link off state
Date: Thu, 4 Feb 2016 11:45:26 +0200 [thread overview]
Message-ID: <1454579128-2682-13-git-send-email-ygardi@codeaurora.org> (raw)
In-Reply-To: <1454579128-2682-1-git-send-email-ygardi@codeaurora.org>
Currently when we try to put the link in off/disabled state during
suspend, it seems link is not being kept in low power mode.
This patch fixes the issue by putting the link in hibern8 first
(so device also puts the link in low power mode) and then stop the
host controller.
Signed-off-by: Subhash Jadavani <subhashj@codeaurora.org>
Signed-off-by: Yaniv Gardi <ygardi@codeaurora.org>
---
drivers/scsi/ufs/ufshcd.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index 3c550e2..f7a699c2 100644
--- a/drivers/scsi/ufs/ufshcd.c
+++ b/drivers/scsi/ufs/ufshcd.c
@@ -5619,6 +5619,16 @@ static int ufshcd_link_state_transition(struct ufs_hba *hba,
(!check_for_bkops || (check_for_bkops &&
!hba->auto_bkops_enabled))) {
/*
+ * Let's make sure that link is in low power mode, we are doing
+ * this currently by putting the link in Hibern8. Otherway to
+ * put the link in low power mode is to send the DME end point
+ * to device and then send the DME reset command to local
+ * unipro. But putting the link in hibern8 is much faster.
+ */
+ ret = ufshcd_uic_hibern8_enter(hba);
+ if (ret)
+ goto out;
+ /*
* Change controller state to "reset state" which
* should also put the link in off/reset state
*/
--
1.8.5.2
--
QUALCOMM ISRAEL, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
next prev parent reply other threads:[~2016-02-04 9:45 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-04 9:45 [v3 00/14] add fixes, device quirks, error recovery, Yaniv Gardi
2016-02-04 9:45 ` [v3 01/14] scsi: ufs-qcom: add number of lanes per direction Yaniv Gardi
2016-02-04 9:45 ` [v3 02/14] scsi: ufs: avoid spurious UFS host controller interrupts Yaniv Gardi
2016-02-04 9:45 ` [v3 03/14] scsi: ufs: implement scsi host timeout handler Yaniv Gardi
2016-02-04 9:45 ` [v3 04/14] scsi: ufs: verify hba controller hce reg value Yaniv Gardi
2016-02-04 9:45 ` [v3 05/14] scsi: ufs: separate device and host quirks Yaniv Gardi
2016-02-04 9:45 ` [v3 06/14] scsi: ufs: disable vccq if it's not needed by UFS device Yaniv Gardi
2016-02-04 9:45 ` [v3 07/14] scsi: ufs: make error handling bit faster Yaniv Gardi
2016-02-04 9:45 ` [v3 08/14] scsi: ufs: add error recovery after DL NAC error Yaniv Gardi
2016-02-04 9:45 ` [v3 09/14] scsi: ufs: add retry for query descriptors Yaniv Gardi
2016-02-04 9:45 ` [v3 10/14] scsi: ufs: handle non spec compliant bkops behaviour by device Yaniv Gardi
2016-02-04 9:45 ` [v3 11/14] scsi: ufs: tune UniPro parameters to optimize hibern8 exit time Yaniv Gardi
2016-02-04 9:45 ` Yaniv Gardi [this message]
2016-02-04 9:45 ` [v3 13/14] scsi: ufs: add device quirk delay before putting UFS rails in LPM Yaniv Gardi
2016-02-04 9:45 ` [v3 14/14] scsi: ufs-qcom: set PA_Local_TX_LCC_Enable before link startup Yaniv Gardi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1454579128-2682-13-git-send-email-ygardi@codeaurora.org \
--to=ygardi@codeaurora.org \
--cc=JBottomley@odin.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=hch@infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi-owner@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=pebolle@tiscali.nl \
--cc=robherring2@gmail.com \
--cc=santoshsy@gmail.com \
--cc=subhashj@codeaurora.org \
--cc=vinholikatti@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).