diff options
author | Zou Wei <zou_wei@huawei.com> | 2021-11-25 14:38:09 +0800 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2021-12-02 09:46:33 +0530 |
commit | b2b56de9faaf19c829ede5cf56918b3793219971 (patch) | |
tree | 0458bbcd856bdc4fd49915ee6ad51347bd03abe7 /drivers/phy/intel | |
parent | 77ba6e7ffbd8b0afe3e475629d5fcb52e7447405 (diff) | |
download | lwn-b2b56de9faaf19c829ede5cf56918b3793219971.tar.gz lwn-b2b56de9faaf19c829ede5cf56918b3793219971.zip |
phy: intel: Remove redundant dev_err call in thunderbay_emmc_phy_probe()
There is a error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zou Wei <zou_wei@huawei.com>
Link: https://lore.kernel.org/r/1637822289-24534-1-git-send-email-zou_wei@huawei.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/phy/intel')
-rw-r--r-- | drivers/phy/intel/phy-intel-thunderbay-emmc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/phy/intel/phy-intel-thunderbay-emmc.c b/drivers/phy/intel/phy-intel-thunderbay-emmc.c index 2d6ea84492f2..593f6970b81e 100644 --- a/drivers/phy/intel/phy-intel-thunderbay-emmc.c +++ b/drivers/phy/intel/phy-intel-thunderbay-emmc.c @@ -472,10 +472,8 @@ static int thunderbay_emmc_phy_probe(struct platform_device *pdev) res = platform_get_resource(pdev, IORESOURCE_MEM, 0); tbh_phy->reg_base = devm_ioremap_resource(&pdev->dev, res); - if (IS_ERR(tbh_phy->reg_base)) { - dev_err(&pdev->dev, "region map failed\n"); + if (IS_ERR(tbh_phy->reg_base)) return PTR_ERR(tbh_phy->reg_base); - } tbh_phy->phy_power_sts = PHY_UNINITIALIZED; id = of_match_node(thunderbay_emmc_phy_of_match, pdev->dev.of_node); |