Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
IronPort-SDR: 2aoHHjpSqELSIDlmRrcDH67X0srwgmHXuCOkgwx/Q2BgmbRau61FoYOmbvedsMTahxQHIHo5C7
 U9QWi7Zi6C5cKut2RvI+PSilRTEPBNDxPNex4yd/YvssA18MsI+3DzVuqnuZYtEqTL7tOI3KSd
 nZG9MmYglFnNCqXtZ1yFX2ig6A+T1iVcT4hpYK57irjtCUSoPmVtk1f2G1j2V1ZHrMfCiOoIYu
 d2/rfMttOl8cX1NiGmuqXHHuRQWQJ4XmnBepzsdUArjufFou1xtfl5TkBjWwQ2EEUcdPiHIkAy
 WuymkWQ4o2hAb7t2xYDypRV2
IronPort-SDR: nSFiIRDyYFAN9/vvxxeJNSpo30SxjZK7lCVRG1DWR3g9y0QpL1w9M74t9nj1rT43lFZcWdmhyo
 oaGHjmIAvToE9JPi3mDtn9AYKxpsC3V7soW3qZcnoFgRU0IAeHhNwgGT5pNOHjNvRf4E0i5RDg
 /WyBx86u+UiIJAcUpsRd9wbG2jOi0j9pCJ+TxG3ETzNoOief9AB6eIo7cD10d9z++JztyQPOlA
 5crQpzu346pasEpZw+ZduoGcCNIHIXJf5YrKJMrTksgBOQJpLH0JY2dy1SlFl6mwwiZwCYYZI9
 o3s=
WDCIronportException: Internal
From:   Arthur Simchaev <Arthur.Simchaev@wdc.com>
To:     martin.petersen@oracle.com
Cc:     -cc=avri.altman@wdc.com, Avi.Shchislowski@wdc.com,
        beanhuo@micron.com, linux-scsi@vger.kernel.org,
        linux-kernel@vger.kernel.org, bvanassche@acm.org,
        Arthur Simchaev <Arthur.Simchaev@wdc.com>
Subject: [PATCH v2] scsi: ufs: Remove redundant function definitions from ufshcd.h
Date:   Wed, 21 Sep 2022 14:58:05 +0300
Message-Id: <1663761485-2532-1-git-send-email-Arthur.Simchaev@wdc.com>
Precedence: bulk

v1->v2: Remove redundant comment, modify commit message

This patch removes Query-Request API functions declaration from
include/ufs/ufshcd.h. Those declarations solely to
ufs core private header.

Signed-off-by: Arthur Simchaev <Arthur.Simchaev@wdc.com>
Reviewed-by: Bean Huo <beanhuo@micron.com>
---
 drivers/ufs/core/ufshcd-priv.h |  5 +++++
 include/ufs/ufshcd.h           | 20 --------------------
 2 files changed, 5 insertions(+), 20 deletions(-)

diff --git a/drivers/ufs/core/ufshcd-priv.h b/drivers/ufs/core/ufshcd-priv.h
index d00dba1..f68ca33 100644
--- a/drivers/ufs/core/ufshcd-priv.h
+++ b/drivers/ufs/core/ufshcd-priv.h
@@ -42,6 +42,11 @@ static inline void ufs_hwmon_remove(struct ufs_hba *hba) {}
 static inline void ufs_hwmon_notify_event(struct ufs_hba *hba, u8 ee_mask) {}
 #endif
 
+int ufshcd_query_descriptor_retry(struct ufs_hba *hba,
+				  enum query_opcode opcode,
+				  enum desc_idn idn, u8 index,
+				  u8 selector,
+				  u8 *desc_buf, int *buf_len);
 int ufshcd_read_desc_param(struct ufs_hba *hba,
 			   enum desc_idn desc_id,
 			   int desc_index,
diff --git a/include/ufs/ufshcd.h b/include/ufs/ufshcd.h
index 24c97e0..9f28349 100644
--- a/include/ufs/ufshcd.h
+++ b/include/ufs/ufshcd.h
@@ -1175,26 +1175,6 @@ static inline int ufshcd_disable_host_tx_lcc(struct ufs_hba *hba)
 	return ufshcd_dme_set(hba, UIC_ARG_MIB(PA_LOCAL_TX_LCC_ENABLE), 0);
 }
 
-/* Expose Query-Request API */
-int ufshcd_query_descriptor_retry(struct ufs_hba *hba,
-				  enum query_opcode opcode,
-				  enum desc_idn idn, u8 index,
-				  u8 selector,
-				  u8 *desc_buf, int *buf_len);
-int ufshcd_read_desc_param(struct ufs_hba *hba,
-			   enum desc_idn desc_id,
-			   int desc_index,
-			   u8 param_offset,
-			   u8 *param_read_buf,
-			   u8 param_size);
-int ufshcd_query_attr_retry(struct ufs_hba *hba, enum query_opcode opcode,
-			    enum attr_idn idn, u8 index, u8 selector,
-			    u32 *attr_val);
-int ufshcd_query_attr(struct ufs_hba *hba, enum query_opcode opcode,
-		      enum attr_idn idn, u8 index, u8 selector, u32 *attr_val);
-int ufshcd_query_flag(struct ufs_hba *hba, enum query_opcode opcode,
-	enum flag_idn idn, u8 index, bool *flag_res);
-
 void ufshcd_auto_hibern8_enable(struct ufs_hba *hba);
 void ufshcd_auto_hibern8_update(struct ufs_hba *hba, u32 ahit);
 void ufshcd_fixup_dev_quirks(struct ufs_hba *hba,
-- 
2.7.4