From: Bean Huo <bean...@micron.com>

Delete ufshcd_read_desc(). Instead, let caller directly call
ufshcd_read_desc_param().

Signed-off-by: Bean Huo <bean...@micron.com>
Reviewed-by: Avri Altman <avri.alt...@wdc.com>
Reviewed-by: Bart van Assche <bvanass...@acm.org>
Reviewed-by: Stanley Chu <stanley....@mediatek.com>
---
 drivers/scsi/ufs/ufshcd.c | 27 ++++++++-------------------
 1 file changed, 8 insertions(+), 19 deletions(-)

diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index 2cf077ab9dfd..c18c2aadbe14 100644
--- a/drivers/scsi/ufs/ufshcd.c
+++ b/drivers/scsi/ufs/ufshcd.c
@@ -3221,16 +3221,6 @@ int ufshcd_read_desc_param(struct ufs_hba *hba,
        return ret;
 }
 
-static inline int ufshcd_read_desc(struct ufs_hba *hba,
-                                  enum desc_idn desc_id,
-                                  int desc_index,
-                                  void *buf,
-                                  u32 size)
-{
-       return ufshcd_read_desc_param(hba, desc_id, desc_index, 0, buf, size);
-}
-
-
 /**
  * struct uc_string_id - unicode string
  *
@@ -3278,9 +3268,8 @@ int ufshcd_read_string_desc(struct ufs_hba *hba, u8 
desc_index,
        if (!uc_str)
                return -ENOMEM;
 
-       ret = ufshcd_read_desc(hba, QUERY_DESC_IDN_STRING,
-                              desc_index, uc_str,
-                              QUERY_DESC_MAX_SIZE);
+       ret = ufshcd_read_desc_param(hba, QUERY_DESC_IDN_STRING, desc_index, 0,
+                                    (u8 *)uc_str, QUERY_DESC_MAX_SIZE);
        if (ret < 0) {
                dev_err(hba->dev, "Reading String Desc failed after %d retries. 
err = %d\n",
                        QUERY_REQ_RETRIES, ret);
@@ -6711,8 +6700,8 @@ static void ufshcd_set_active_icc_lvl(struct ufs_hba *hba)
        if (!desc_buf)
                return;
 
-       ret = ufshcd_read_desc(hba, QUERY_DESC_IDN_POWER, 0,
-                       desc_buf, buff_len);
+       ret = ufshcd_read_desc_param(hba, QUERY_DESC_IDN_POWER, 0, 0,
+                                    desc_buf, buff_len);
        if (ret) {
                dev_err(hba->dev,
                        "%s: Failed reading power descriptor.len = %d ret = %d",
@@ -6913,8 +6902,8 @@ static int ufs_get_device_desc(struct ufs_hba *hba)
                goto out;
        }
 
-       err = ufshcd_read_desc(hba, QUERY_DESC_IDN_DEVICE, 0, desc_buf,
-                       hba->desc_size.dev_desc);
+       err = ufshcd_read_desc_param(hba, QUERY_DESC_IDN_DEVICE, 0, 0, desc_buf,
+                                    hba->desc_size.dev_desc);
        if (err) {
                dev_err(hba->dev, "%s: Failed reading Device Desc. err = %d\n",
                        __func__, err);
@@ -7196,8 +7185,8 @@ static int ufshcd_device_geo_params_init(struct ufs_hba 
*hba)
                goto out;
        }
 
-       err = ufshcd_read_desc(hba, QUERY_DESC_IDN_GEOMETRY, 0,
-                       desc_buf, buff_len);
+       err = ufshcd_read_desc_param(hba, QUERY_DESC_IDN_GEOMETRY, 0, 0,
+                                    desc_buf, buff_len);
        if (err) {
                dev_err(hba->dev, "%s: Failed reading Geometry Desc. err = 
%d\n",
                                __func__, err);
-- 
2.17.1

Reply via email to