Skip to contentSkip to conversation

ufs: Add support for clock gating

DISCUSSION Dan Carpenter 1 participant 1 message Oct 2, 2014 SCSI

lore
Share & tools
lore
#0 lore raw

Hello Sahitya Tummala,

The patch 1ab27c9cf8b6: "ufs: Add support for clock gating" from Sep 25, 2014, leads to the following static checker warning:

	drivers/scsi/ufs/ufshcd.c:4474 __ufshcd_setup_clocks()
	warn: we tested 'ret' before and it was 'false'

drivers/scsi/ufs/ufshcd.c
4467 ret = hba->vops->setup_clocks(hba, on);
4468 out:
4469 if (ret) {
4470 list_for_each_entry(clki, head, list) {
4471 if (!IS_ERR_OR_NULL(clki->clk) && clki->enabled)
4472 clk_disable_unprepare(clki->clk);
4473 }

  4474          } else if (!ret && on) {
                           ^^^^

Not needed.

  4475                  spin_lock_irqsave(hba->host->host_lock, flags);
  4476                  hba->clk_gating.state = CLKS_ON;
  4477                  spin_unlock_irqrestore(hba->host->host_lock, flags);

4478 }
4479 return ret;
4480 }

regards,
dan carpenter