Reviewed-by: Gilad Broner <gbro...@codeaurora.org>
> The dme_peer get/set attribute commands are prone to errors, therefore
> we add three retries for the UIC command sending.
> Error code returned from ufshcd_send_uic_cmd() is checked, and unless
> it was successful or the retries have finished, another command will be
> sent.
>
> Signed-off-by: Lee Susman <lsus...@codeaurora.org>
> Signed-off-by: Yaniv Gardi <yga...@codeaurora.org>
>
> ---
> drivers/scsi/ufs/ufshcd.c | 40 +++++++++++++++++++++++++++++-----------
> 1 file changed, 29 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
> index 8a04691..5005d12 100644
> --- a/drivers/scsi/ufs/ufshcd.c
> +++ b/drivers/scsi/ufs/ufshcd.c
> @@ -70,6 +70,9 @@
> /* Task management command timeout */
> #define TM_CMD_TIMEOUT 100 /* msecs */
>
> +/* maximum number of retries for a general UIC command */
> +#define UFS_UIC_COMMAND_RETRIES 3
> +
> /* maximum number of link-startup retries */
> #define DME_LINKSTARTUP_RETRIES 3
>
> @@ -2185,6 +2188,7 @@ int ufshcd_dme_set_attr(struct ufs_hba *hba, u32
> attr_sel,
> };
> const char *set = action[!!peer];
> int ret;
> + int retries = UFS_UIC_COMMAND_RETRIES;
>
> uic_cmd.command = peer ?
> UIC_CMD_DME_PEER_SET : UIC_CMD_DME_SET;
> @@ -2192,10 +2196,18 @@ int ufshcd_dme_set_attr(struct ufs_hba *hba, u32
> attr_sel,
> uic_cmd.argument2 = UIC_ARG_ATTR_TYPE(attr_set);
> uic_cmd.argument3 = mib_val;
>
> - ret = ufshcd_send_uic_cmd(hba, &uic_cmd);
> - if (ret)
> - dev_err(hba->dev, "%s: attr-id 0x%x val 0x%x error code %d\n",
> - set, UIC_GET_ATTR_ID(attr_sel), mib_val, ret);
> + do {
> + /* for peer attributes we retry upon failure */
> + ret = ufshcd_send_uic_cmd(hba, &uic_cmd);
> + if (ret)
> + dev_dbg(hba->dev, "%s: attr-id 0x%x val 0x%x error code
> %d\n",
> + set, UIC_GET_ATTR_ID(attr_sel), mib_val, ret);
> + } while (ret && peer && --retries);
> +
> + if (!retries)
> + dev_err(hba->dev, "%s: attr-id 0x%x val 0x%x failed %d
> retries\n",
> + set, UIC_GET_ATTR_ID(attr_sel), mib_val,
> + retries);
>
> return ret;
> }
> @@ -2220,6 +2232,7 @@ int ufshcd_dme_get_attr(struct ufs_hba *hba, u32
> attr_sel,
> };
> const char *get = action[!!peer];
> int ret;
> + int retries = UFS_UIC_COMMAND_RETRIES;
> struct ufs_pa_layer_attr orig_pwr_info;
> struct ufs_pa_layer_attr temp_pwr_info;
> bool pwr_mode_change = false;
> @@ -2250,14 +2263,19 @@ int ufshcd_dme_get_attr(struct ufs_hba *hba, u32
> attr_sel,
> UIC_CMD_DME_PEER_GET : UIC_CMD_DME_GET;
> uic_cmd.argument1 = attr_sel;
>
> - ret = ufshcd_send_uic_cmd(hba, &uic_cmd);
> - if (ret) {
> - dev_err(hba->dev, "%s: attr-id 0x%x error code %d\n",
> - get, UIC_GET_ATTR_ID(attr_sel), ret);
> - goto out;
> - }
> + do {
> + /* for peer attributes we retry upon failure */
> + ret = ufshcd_send_uic_cmd(hba, &uic_cmd);
> + if (ret)
> + dev_dbg(hba->dev, "%s: attr-id 0x%x error code %d\n",
> + get, UIC_GET_ATTR_ID(attr_sel), ret);
> + } while (ret && peer && --retries);
> +
> + if (!retries)
> + dev_err(hba->dev, "%s: attr-id 0x%x failed %d retries\n",
> + get, UIC_GET_ATTR_ID(attr_sel), retries);
>
> - if (mib_val)
> + if (mib_val && !ret)
> *mib_val = uic_cmd.argument3;
>
> if (peer && (hba->quirks & UFSHCD_QUIRK_DME_PEER_ACCESS_AUTO_MODE)
> --
> 1.8.5.2
>
> --
> QUALCOMM ISRAEL, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Qualcomm Israel, on behalf of Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
