[PATCH v2 6/7] drivers: ufs: qcom: Initialize and enable clocks before hardware access

Neha Malcom Francis n-francis at ti.com
Mon May 4 13:07:39 CEST 2026


On Mon, 27 Apr 2026 14:56:10 +0530, Balaji Selvanathan <balaji.selvanathan at oss.qualcomm.com> wrote:
> diff --git a/drivers/ufs/ufs-qcom.c b/drivers/ufs/ufs-qcom.c
> index dc40ee62daf..ae33f62fbee 100644
> --- a/drivers/ufs/ufs-qcom.c
> +++ b/drivers/ufs/ufs-qcom.c
> @@ -557,10 +547,45 @@ static void ufs_qcom_dev_ref_clk_ctrl(struct ufs_hba *hba, bool enable)
> [ ... skip 20 lines ... ]
> +
> +	rate = clk_set_rate(&clk, max_freq);
> +	if (rate < 0) {
> +		dev_err(dev, "Failed to set core_clk_unipro rate to %u Hz: %ld\n",
> +			max_freq, rate);
> +	}

Was it intentional to proceed with init despite core clock set_rate failure?

-- 
Neha Malcom Francis <n-francis at ti.com>


More information about the U-Boot mailing list