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
