linux-kernel - Re: [PATCH v6 16/17] scsi: ufs-qcom: enable/disable the device ref clock

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 7 Mar 2016 08:55:21 -0000
From:	"Gilad Broner" <gbroner@...eaurora.org>
To:	"Yaniv Gardi" <ygardi@...eaurora.org>
Cc:	james.bottomley@...senpartnership.com,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
	linux-arm-msm@...r.kernel.org, santoshsy@...il.com,
	linux-scsi-owner@...r.kernel.org, ygardi@...eaurora.org,
	"Vinayak Holikatti" <vinholikatti@...il.com>,
	"James E.J. Bottomley" <jbottomley@...n.com>,
	"Martin K. Petersen" <martin.petersen@...cle.com>
Subject: Re: [PATCH v6 16/17] scsi: ufs-qcom: enable/disable the device ref
 clock

Reviewed-by: Gilad Broner <gbroner@...eaurora.org>

> This change enables the device ref clock before changing to HS mode
> and disables it if entered to PWM mode.
>
> Signed-off-by: Yaniv Gardi <ygardi@...eaurora.org>
>
> ---
>  drivers/scsi/ufs/ufs-qcom.c | 12 ++++++++++++
>  drivers/scsi/ufs/ufshcd.h   |  9 +++++++++
>  2 files changed, 21 insertions(+)
>
> diff --git a/drivers/scsi/ufs/ufs-qcom.c b/drivers/scsi/ufs/ufs-qcom.c
> index 966bacf..849fcf3 100644
> --- a/drivers/scsi/ufs/ufs-qcom.c
> +++ b/drivers/scsi/ufs/ufs-qcom.c
> @@ -980,6 +980,10 @@ static int ufs_qcom_pwr_change_notify(struct ufs_hba
> *hba,
>  			goto out;
>  		}
>
> +		/* enable the device ref clock before changing to HS mode */
> +		if (!ufshcd_is_hs_mode(&hba->pwr_info) &&
> +			ufshcd_is_hs_mode(dev_req_params))
> +			ufs_qcom_dev_ref_clk_ctrl(host, true);
>  		break;
>  	case POST_CHANGE:
>  		if (ufs_qcom_cfg_timers(hba, dev_req_params->gear_rx,
> @@ -1007,6 +1011,11 @@ static int ufs_qcom_pwr_change_notify(struct
> ufs_hba *hba,
>  		memcpy(&host->dev_req_params,
>  				dev_req_params, sizeof(*dev_req_params));
>  		ufs_qcom_update_bus_bw_vote(host);
> +
> +		/* disable the device ref clock if entered PWM mode */
> +		if (ufshcd_is_hs_mode(&hba->pwr_info) &&
> +			!ufshcd_is_hs_mode(dev_req_params))
> +			ufs_qcom_dev_ref_clk_ctrl(host, false);
>  		break;
>  	default:
>  		ret = -EINVAL;
> @@ -1108,6 +1117,9 @@ static int ufs_qcom_setup_clocks(struct ufs_hba
> *hba, bool on)
>  			ufs_qcom_phy_disable_iface_clk(host->generic_phy);
>  			goto out;
>  		}
> +		/* enable the device ref clock for HS mode*/
> +		if (ufshcd_is_hs_mode(&hba->pwr_info))
> +			ufs_qcom_dev_ref_clk_ctrl(host, true);
>  		vote = host->bus_vote.saved_vote;
>  		if (vote == host->bus_vote.min_bw_vote)
>  			ufs_qcom_update_bus_bw_vote(host);
> diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h
> index 41d9bfd..4bb6566 100644
> --- a/drivers/scsi/ufs/ufshcd.h
> +++ b/drivers/scsi/ufs/ufshcd.h
> @@ -54,6 +54,7 @@
>  #include <linux/clk.h>
>  #include <linux/completion.h>
>  #include <linux/regulator/consumer.h>
> +#include "unipro.h"
>
>  #include <asm/irq.h>
>  #include <asm/byteorder.h>
> @@ -689,6 +690,14 @@ static inline int ufshcd_dme_peer_get(struct ufs_hba
> *hba,
>
>  int ufshcd_read_device_desc(struct ufs_hba *hba, u8 *buf, u32 size);
>
> +static inline bool ufshcd_is_hs_mode(struct ufs_pa_layer_attr *pwr_info)
> +{
> +	return (pwr_info->pwr_rx == FAST_MODE ||
> +		pwr_info->pwr_rx == FASTAUTO_MODE) &&
> +		(pwr_info->pwr_tx == FAST_MODE ||
> +		pwr_info->pwr_tx == FASTAUTO_MODE);
> +}
> +
>  #define ASCII_STD true
>
>  int ufshcd_read_string_desc(struct ufs_hba *hba, int desc_index, u8 *buf,
> --
> 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 majordomo@...r.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