re: ufs: Add support for clock gating

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux