* [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 [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
all inboxes | Powered by JetHome®