diff mbox series

[v5,6/7] scsi: ufs: Cleanup WB buffer flush toggle implementation

Message ID 20201215230519.15158-7-huobean@gmail.com
State Superseded
Headers show
Series Several changes for UFS WriteBooster | expand

Commit Message

Bean Huo Dec. 15, 2020, 11:05 p.m. UTC
From: Bean Huo <beanhuo@micron.com>

Delete ufshcd_wb_buf_flush_enable() and ufshcd_wb_buf_flush_disable(),
move the implementation into ufshcd_wb_toggle_flush().

Signed-off-by: Bean Huo <beanhuo@micron.com>
---
 drivers/scsi/ufs/ufshcd.c | 66 +++++++++++++--------------------------
 1 file changed, 21 insertions(+), 45 deletions(-)

Comments

Stanley Chu Dec. 16, 2020, 4:17 a.m. UTC | #1
On Wed, 2020-12-16 at 00:05 +0100, Bean Huo wrote:
> From: Bean Huo <beanhuo@micron.com>

> 

> Delete ufshcd_wb_buf_flush_enable() and ufshcd_wb_buf_flush_disable(),

> move the implementation into ufshcd_wb_toggle_flush().

> 

> Signed-off-by: Bean Huo <beanhuo@micron.com>

> ---


Reviewed-by: Stanley Chu <stanley.chu@mediatek.com>
Can Guo Dec. 22, 2020, 6:14 a.m. UTC | #2
On 2020-12-16 07:05, Bean Huo wrote:
> From: Bean Huo <beanhuo@micron.com>

> 

> Delete ufshcd_wb_buf_flush_enable() and ufshcd_wb_buf_flush_disable(),

> move the implementation into ufshcd_wb_toggle_flush().

> 


Reviewed-by: Can Guo <cang@codeaurora.org>


> Signed-off-by: Bean Huo <beanhuo@micron.com>

> ---

>  drivers/scsi/ufs/ufshcd.c | 66 +++++++++++++--------------------------

>  1 file changed, 21 insertions(+), 45 deletions(-)

> 

> diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c

> index 466a85051d54..ba8298f0d017 100644

> --- a/drivers/scsi/ufs/ufshcd.c

> +++ b/drivers/scsi/ufs/ufshcd.c

> @@ -244,10 +244,8 @@ static int ufshcd_setup_vreg(struct ufs_hba *hba, 

> bool on);

>  static inline int ufshcd_config_vreg_hpm(struct ufs_hba *hba,

>  					 struct ufs_vreg *vreg);

>  static int ufshcd_try_to_abort_task(struct ufs_hba *hba, int tag);

> -static int ufshcd_wb_buf_flush_enable(struct ufs_hba *hba);

> -static int ufshcd_wb_buf_flush_disable(struct ufs_hba *hba);

>  static int ufshcd_wb_toggle_flush_during_h8(struct ufs_hba *hba, bool 

> set);

> -static inline void ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool 

> enable);

> +static inline int ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool 

> enable);

>  static void ufshcd_hba_vreg_set_lpm(struct ufs_hba *hba);

>  static void ufshcd_hba_vreg_set_hpm(struct ufs_hba *hba);

> 

> @@ -5398,60 +5396,38 @@ static int

> ufshcd_wb_toggle_flush_during_h8(struct ufs_hba *hba, bool set)

>  				index, NULL);

>  }

> 

> -static inline void ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool 

> enable)

> -{

> -	if (hba->quirks & UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL)

> -		return;

> -

> -	if (enable)

> -		ufshcd_wb_buf_flush_enable(hba);

> -	else

> -		ufshcd_wb_buf_flush_disable(hba);

> -

> -}

> -

> -static int ufshcd_wb_buf_flush_enable(struct ufs_hba *hba)

> +static inline int ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool 

> enable)

>  {

>  	int ret;

>  	u8 index;

> +	enum query_opcode opcode;

> 

> -	if (!ufshcd_is_wb_allowed(hba) || hba->dev_info.wb_buf_flush_enabled)

> +	if (hba->quirks & UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL)

>  		return 0;

> 

> -	index = ufshcd_wb_get_query_index(hba);

> -	ret = ufshcd_query_flag_retry(hba, UPIU_QUERY_OPCODE_SET_FLAG,

> -				      QUERY_FLAG_IDN_WB_BUFF_FLUSH_EN,

> -				      index, NULL);

> -	if (ret)

> -		dev_err(hba->dev, "%s WB - buf flush enable failed %d\n",

> -			__func__, ret);

> -	else

> -		hba->dev_info.wb_buf_flush_enabled = true;

> -

> -	dev_dbg(hba->dev, "WB - Flush enabled: %d\n", ret);

> -	return ret;

> -}

> -

> -static int ufshcd_wb_buf_flush_disable(struct ufs_hba *hba)

> -{

> -	int ret;

> -	u8 index;

> -

> -	if (!ufshcd_is_wb_allowed(hba) || 

> !hba->dev_info.wb_buf_flush_enabled)

> +	if (!ufshcd_is_wb_allowed(hba) ||

> +	    hba->dev_info.wb_buf_flush_enabled == enable)

>  		return 0;

> 

> +	if (enable)

> +		opcode = UPIU_QUERY_OPCODE_SET_FLAG;

> +	else

> +		opcode = UPIU_QUERY_OPCODE_CLEAR_FLAG;

> +

>  	index = ufshcd_wb_get_query_index(hba);

> -	ret = ufshcd_query_flag_retry(hba, UPIU_QUERY_OPCODE_CLEAR_FLAG,

> -				      QUERY_FLAG_IDN_WB_BUFF_FLUSH_EN,

> -				      index, NULL);

> +	ret = ufshcd_query_flag_retry(hba, opcode,

> +				      QUERY_FLAG_IDN_WB_BUFF_FLUSH_EN, index,

> +				      NULL);

>  	if (ret) {

> -		dev_warn(hba->dev, "%s: WB - buf flush disable failed %d\n",

> -			 __func__, ret);

> -	} else {

> -		hba->dev_info.wb_buf_flush_enabled = false;

> -		dev_dbg(hba->dev, "WB - Flush disabled: %d\n", ret);

> +		dev_err(hba->dev, "%s WB-Buf Flush %s failed %d\n", __func__,

> +			enable ? "enable" : "disable", ret);

> +		goto out;

>  	}

> 

> +	hba->dev_info.wb_buf_flush_enabled = enable;

> +

> +	dev_dbg(hba->dev, "WB-Buf Flush %s\n", enable ? "enabled" : 

> "disabled");

> +out:

>  	return ret;

>  }
diff mbox series

Patch

diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index 466a85051d54..ba8298f0d017 100644
--- a/drivers/scsi/ufs/ufshcd.c
+++ b/drivers/scsi/ufs/ufshcd.c
@@ -244,10 +244,8 @@  static int ufshcd_setup_vreg(struct ufs_hba *hba, bool on);
 static inline int ufshcd_config_vreg_hpm(struct ufs_hba *hba,
 					 struct ufs_vreg *vreg);
 static int ufshcd_try_to_abort_task(struct ufs_hba *hba, int tag);
-static int ufshcd_wb_buf_flush_enable(struct ufs_hba *hba);
-static int ufshcd_wb_buf_flush_disable(struct ufs_hba *hba);
 static int ufshcd_wb_toggle_flush_during_h8(struct ufs_hba *hba, bool set);
-static inline void ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool enable);
+static inline int ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool enable);
 static void ufshcd_hba_vreg_set_lpm(struct ufs_hba *hba);
 static void ufshcd_hba_vreg_set_hpm(struct ufs_hba *hba);
 
@@ -5398,60 +5396,38 @@  static int ufshcd_wb_toggle_flush_during_h8(struct ufs_hba *hba, bool set)
 				index, NULL);
 }
 
-static inline void ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool enable)
-{
-	if (hba->quirks & UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL)
-		return;
-
-	if (enable)
-		ufshcd_wb_buf_flush_enable(hba);
-	else
-		ufshcd_wb_buf_flush_disable(hba);
-
-}
-
-static int ufshcd_wb_buf_flush_enable(struct ufs_hba *hba)
+static inline int ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool enable)
 {
 	int ret;
 	u8 index;
+	enum query_opcode opcode;
 
-	if (!ufshcd_is_wb_allowed(hba) || hba->dev_info.wb_buf_flush_enabled)
+	if (hba->quirks & UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL)
 		return 0;
 
-	index = ufshcd_wb_get_query_index(hba);
-	ret = ufshcd_query_flag_retry(hba, UPIU_QUERY_OPCODE_SET_FLAG,
-				      QUERY_FLAG_IDN_WB_BUFF_FLUSH_EN,
-				      index, NULL);
-	if (ret)
-		dev_err(hba->dev, "%s WB - buf flush enable failed %d\n",
-			__func__, ret);
-	else
-		hba->dev_info.wb_buf_flush_enabled = true;
-
-	dev_dbg(hba->dev, "WB - Flush enabled: %d\n", ret);
-	return ret;
-}
-
-static int ufshcd_wb_buf_flush_disable(struct ufs_hba *hba)
-{
-	int ret;
-	u8 index;
-
-	if (!ufshcd_is_wb_allowed(hba) || !hba->dev_info.wb_buf_flush_enabled)
+	if (!ufshcd_is_wb_allowed(hba) ||
+	    hba->dev_info.wb_buf_flush_enabled == enable)
 		return 0;
 
+	if (enable)
+		opcode = UPIU_QUERY_OPCODE_SET_FLAG;
+	else
+		opcode = UPIU_QUERY_OPCODE_CLEAR_FLAG;
+
 	index = ufshcd_wb_get_query_index(hba);
-	ret = ufshcd_query_flag_retry(hba, UPIU_QUERY_OPCODE_CLEAR_FLAG,
-				      QUERY_FLAG_IDN_WB_BUFF_FLUSH_EN,
-				      index, NULL);
+	ret = ufshcd_query_flag_retry(hba, opcode,
+				      QUERY_FLAG_IDN_WB_BUFF_FLUSH_EN, index,
+				      NULL);
 	if (ret) {
-		dev_warn(hba->dev, "%s: WB - buf flush disable failed %d\n",
-			 __func__, ret);
-	} else {
-		hba->dev_info.wb_buf_flush_enabled = false;
-		dev_dbg(hba->dev, "WB - Flush disabled: %d\n", ret);
+		dev_err(hba->dev, "%s WB-Buf Flush %s failed %d\n", __func__,
+			enable ? "enable" : "disable", ret);
+		goto out;
 	}
 
+	hba->dev_info.wb_buf_flush_enabled = enable;
+
+	dev_dbg(hba->dev, "WB-Buf Flush %s\n", enable ? "enabled" : "disabled");
+out:
 	return ret;
 }