On 8/16/23 8:35 PM, Marek Vasut wrote:
Use utp_transfer_req_desc pointer to reference to utrdl queue
instead of referencing the queue directly. This makes the code
more consistent. No functional change.

Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
---
Cc: Bhupesh Sharma <bhupesh.sha...@linaro.org>
Cc: Faiz Abbas <faiz_ab...@ti.com>
---
V2: No change
---
  drivers/ufs/ufs.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/ufs/ufs.c b/drivers/ufs/ufs.c
index da1009e2c14..041caee714f 100644
--- a/drivers/ufs/ufs.c
+++ b/drivers/ufs/ufs.c
@@ -858,7 +858,9 @@ static inline int ufshcd_get_req_rsp(struct utp_upiu_rsp 
*ucd_rsp_ptr)
   */
  static inline int ufshcd_get_tr_ocs(struct ufs_hba *hba)
  {
-       return le32_to_cpu(hba->utrdl->header.dword_2) & MASK_OCS;
+       struct utp_transfer_req_desc *req_desc = hba->utrdl;
+
+       return le32_to_cpu(req_desc->header.dword_2) & MASK_OCS;
  }
static inline int ufshcd_get_rsp_upiu_result(struct utp_upiu_rsp *ucd_rsp_ptr)

Reviewed-by: Bhupesh Sharma <bhupesh.sha...@linaro.org>
Tested-by: Bhupesh Sharma <bhupesh.sha...@linaro.org>

Reply via email to