Re: [PATCH v3 4/6] scsi: ufs: host: Add support for parsing OPP

From: Viresh Kumar
Date: Tue Aug 01 2023 - 05:05:48 EST


On 31-07-23, 22:03, Manivannan Sadhasivam wrote:
> +int ufshcd_opp_config_clks(struct device *dev, struct opp_table *opp_table,
> + struct dev_pm_opp *opp, void *data,
> + bool scaling_down)
> +{
> + struct ufs_hba *hba = dev_get_drvdata(dev);
> + struct list_head *head = &hba->clk_list_head;
> + struct ufs_clk_info *clki;
> + unsigned long freq;
> + u8 idx = 0;
> + int ret;
> +
> + list_for_each_entry(clki, head, list) {
> + if (!IS_ERR_OR_NULL(clki->clk)) {

NULL is considered as a valid clock AFAIR, so you must only be doing
IS_ERR() here I guess.

--
viresh