diff options
author | Vivek Gautam <vivek.gautam@codeaurora.org> | 2016-11-08 13:07:40 +0300 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2016-11-09 02:05:44 +0300 |
commit | fd91ddad2e9a1eab3df63ccd0f60fdfa08dbe452 (patch) | |
tree | 9821fba50fca3c5349135126ace57eec3118ac5b /drivers/phy/phy-qcom-ufs.c | |
parent | fd3e165ac8be7bfbc47318e7b720458a5d5d6227 (diff) | |
download | linux-fd91ddad2e9a1eab3df63ccd0f60fdfa08dbe452.tar.xz |
phy: qcom-ufs: Remove unnecessary BUG_ON
BUG_ON() are not preferred in the driver, plus the variable
on which BUG_ON is asserted is already checked in the code
before passing.
Signed-off-by: Vivek Gautam <vivek.gautam@codeaurora.org>
Reviewed-by: Subhash Jadavani <subhashj@codeaurora.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/phy/phy-qcom-ufs.c')
-rw-r--r-- | drivers/phy/phy-qcom-ufs.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/phy/phy-qcom-ufs.c b/drivers/phy/phy-qcom-ufs.c index 18a5b495ad65..455064cd270d 100644 --- a/drivers/phy/phy-qcom-ufs.c +++ b/drivers/phy/phy-qcom-ufs.c @@ -322,8 +322,6 @@ int ufs_qcom_phy_cfg_vreg(struct phy *phy, struct ufs_qcom_phy *ufs_qcom_phy = get_ufs_qcom_phy(phy); struct device *dev = ufs_qcom_phy->dev; - BUG_ON(!vreg); - if (regulator_count_voltages(reg) > 0) { min_uV = on ? vreg->min_uV : 0; ret = regulator_set_voltage(reg, min_uV, vreg->max_uV); |