* [PATCH] scsi: ufs: ufshpb: Use proper power management API
[not found] <CGME20210902003534epcms2p1937a0f0eeb48a441cb69f5ef13ff8430@epcms2p1>
@ 2021-09-02 0:35 ` Daejun Park
2021-09-14 8:41 ` Avri Altman
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Daejun Park @ 2021-09-02 0:35 UTC (permalink / raw)
To: ALIM AKHTAR, avri.altman, jejb, martin.petersen, Daejun Park,
bvanassche, Keoseong Park, huobean
Cc: linux-scsi, linux-kernel
In ufshpb, pm_runtime_{get,put}_sync() are used to avoid unwanted runtime
suspend during query requests. In commit b294ff3e3449 ("scsi: ufs: core:
Enable power management for wlun") has been modified to use
ufshcd_rpm_{get,put}_sync() APIs.
This patch has been modified to use these APIs in HPB module.
Signed-off-by: Daejun Park <daejun7.park@samsung.com>
---
drivers/scsi/ufs/ufshpb.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/ufs/ufshpb.c b/drivers/scsi/ufs/ufshpb.c
index 02fb51ae8b25..9ea639bf6a59 100644
--- a/drivers/scsi/ufs/ufshpb.c
+++ b/drivers/scsi/ufs/ufshpb.c
@@ -2650,11 +2650,11 @@ static int ufshpb_get_lu_info(struct ufs_hba *hba, int lun,
ufshcd_map_desc_id_to_length(hba, QUERY_DESC_IDN_UNIT, &size);
- pm_runtime_get_sync(hba->dev);
+ ufshcd_rpm_get_sync(hba);
ret = ufshcd_query_descriptor_retry(hba, UPIU_QUERY_OPCODE_READ_DESC,
QUERY_DESC_IDN_UNIT, lun, 0,
desc_buf, &size);
- pm_runtime_put_sync(hba->dev);
+ ufshcd_rpm_put_sync(hba);
if (ret) {
dev_err(hba->dev,
@@ -2877,10 +2877,10 @@ void ufshpb_get_dev_info(struct ufs_hba *hba, u8 *desc_buf)
if (version == HPB_SUPPORT_LEGACY_VERSION)
hpb_dev_info->is_legacy = true;
- pm_runtime_get_sync(hba->dev);
+ ufshcd_rpm_get_sync(hba);
ret = ufshcd_query_attr_retry(hba, UPIU_QUERY_OPCODE_READ_ATTR,
QUERY_ATTR_IDN_MAX_HPB_SINGLE_CMD, 0, 0, &max_hpb_single_cmd);
- pm_runtime_put_sync(hba->dev);
+ ufshcd_rpm_put_sync(hba);
if (ret)
dev_err(hba->dev, "%s: idn: read max size of single hpb cmd query request failed",
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [PATCH] scsi: ufs: ufshpb: Use proper power management API
2021-09-02 0:35 ` [PATCH] scsi: ufs: ufshpb: Use proper power management API Daejun Park
@ 2021-09-14 8:41 ` Avri Altman
2021-09-15 3:29 ` Martin K. Petersen
2021-09-22 4:45 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Avri Altman @ 2021-09-14 8:41 UTC (permalink / raw)
To: daejun7.park, ALIM AKHTAR, jejb, martin.petersen, bvanassche,
Keoseong Park, huobean
Cc: linux-scsi, linux-kernel
> In ufshpb, pm_runtime_{get,put}_sync() are used to avoid unwanted runtime
> suspend during query requests. In commit b294ff3e3449 ("scsi: ufs: core:
> Enable power management for wlun") has been modified to use
> ufshcd_rpm_{get,put}_sync() APIs.
>
> This patch has been modified to use these APIs in HPB module.
>
> Signed-off-by: Daejun Park <daejun7.park@samsung.com>
Reviewed-by: Avri Altman <avri.altman@wdc.com>
> ---
> drivers/scsi/ufs/ufshpb.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/ufs/ufshpb.c b/drivers/scsi/ufs/ufshpb.c
> index 02fb51ae8b25..9ea639bf6a59 100644
> --- a/drivers/scsi/ufs/ufshpb.c
> +++ b/drivers/scsi/ufs/ufshpb.c
> @@ -2650,11 +2650,11 @@ static int ufshpb_get_lu_info(struct ufs_hba *hba,
> int lun,
>
> ufshcd_map_desc_id_to_length(hba, QUERY_DESC_IDN_UNIT, &size);
>
> - pm_runtime_get_sync(hba->dev);
> + ufshcd_rpm_get_sync(hba);
> ret = ufshcd_query_descriptor_retry(hba,
> UPIU_QUERY_OPCODE_READ_DESC,
> QUERY_DESC_IDN_UNIT, lun, 0,
> desc_buf, &size);
> - pm_runtime_put_sync(hba->dev);
> + ufshcd_rpm_put_sync(hba);
>
> if (ret) {
> dev_err(hba->dev,
> @@ -2877,10 +2877,10 @@ void ufshpb_get_dev_info(struct ufs_hba *hba, u8
> *desc_buf)
> if (version == HPB_SUPPORT_LEGACY_VERSION)
> hpb_dev_info->is_legacy = true;
>
> - pm_runtime_get_sync(hba->dev);
> + ufshcd_rpm_get_sync(hba);
> ret = ufshcd_query_attr_retry(hba, UPIU_QUERY_OPCODE_READ_ATTR,
> QUERY_ATTR_IDN_MAX_HPB_SINGLE_CMD, 0, 0,
> &max_hpb_single_cmd);
> - pm_runtime_put_sync(hba->dev);
> + ufshcd_rpm_put_sync(hba);
>
> if (ret)
> dev_err(hba->dev, "%s: idn: read max size of single hpb cmd query
> request failed",
> --
> 2.25.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: ufs: ufshpb: Use proper power management API
2021-09-02 0:35 ` [PATCH] scsi: ufs: ufshpb: Use proper power management API Daejun Park
2021-09-14 8:41 ` Avri Altman
@ 2021-09-15 3:29 ` Martin K. Petersen
2021-09-22 4:45 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2021-09-15 3:29 UTC (permalink / raw)
To: Daejun Park
Cc: ALIM AKHTAR, avri.altman, jejb, martin.petersen, bvanassche,
Keoseong Park, huobean, linux-scsi, linux-kernel
Daejun,
> In ufshpb, pm_runtime_{get,put}_sync() are used to avoid unwanted
> runtime suspend during query requests. In commit b294ff3e3449 ("scsi:
> ufs: core: Enable power management for wlun") has been modified to use
> ufshcd_rpm_{get,put}_sync() APIs.
Applied to 5.16/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: ufs: ufshpb: Use proper power management API
2021-09-02 0:35 ` [PATCH] scsi: ufs: ufshpb: Use proper power management API Daejun Park
2021-09-14 8:41 ` Avri Altman
2021-09-15 3:29 ` Martin K. Petersen
@ 2021-09-22 4:45 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2021-09-22 4:45 UTC (permalink / raw)
To: bvanassche, ALIM AKHTAR, avri.altman, jejb, Keoseong Park,
Daejun Park, huobean
Cc: Martin K . Petersen, linux-kernel, linux-scsi
On Thu, 02 Sep 2021 09:35:34 +0900, Daejun Park wrote:
> In ufshpb, pm_runtime_{get,put}_sync() are used to avoid unwanted runtime
> suspend during query requests. In commit b294ff3e3449 ("scsi: ufs: core:
> Enable power management for wlun") has been modified to use
> ufshcd_rpm_{get,put}_sync() APIs.
>
> This patch has been modified to use these APIs in HPB module.
>
> [...]
Applied to 5.16/scsi-queue, thanks!
[1/1] scsi: ufs: ufshpb: Use proper power management API
https://git.kernel.org/mkp/scsi/c/351b3a849ac7
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-09-22 4:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20210902003534epcms2p1937a0f0eeb48a441cb69f5ef13ff8430@epcms2p1>
2021-09-02 0:35 ` [PATCH] scsi: ufs: ufshpb: Use proper power management API Daejun Park
2021-09-14 8:41 ` Avri Altman
2021-09-15 3:29 ` Martin K. Petersen
2021-09-22 4:45 ` Martin K. Petersen
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).