clk: davinci: Remove redundant dev_err calls
authorWei Yongjun <weiyongjun1@huawei.com>
Thu, 22 Mar 2018 02:11:48 +0000 (02:11 +0000)
committerStephen Boyd <sboyd@kernel.org>
Fri, 23 Mar 2018 16:44:45 +0000 (09:44 -0700)
There is a error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.

Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Reviewed-by: David Lechner <david@lechnology.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/davinci/pll.c
drivers/clk/davinci/psc.c

index 89d30bf95102333d7ec3c5014d763129d2b54d8a..23a24c944f1d2f11aa2c3eba403a77ef0d89d41a 100644 (file)
@@ -814,10 +814,8 @@ static int davinci_pll_probe(struct platform_device *pdev)
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        base = devm_ioremap_resource(dev, res);
-       if (IS_ERR(base)) {
-               dev_err(dev, "ioremap failed\n");
+       if (IS_ERR(base))
                return PTR_ERR(base);
-       }
 
        return pll_init(dev, base);
 }
index 3b0e59dfbdd760e9ea3f85ef24a9519ced859cbf..ff6f4a038e0479db0e224a2576e6effb486ea54c 100644 (file)
@@ -521,10 +521,8 @@ static int davinci_psc_probe(struct platform_device *pdev)
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        base = devm_ioremap_resource(dev, res);
-       if (IS_ERR(base)) {
-               dev_err(dev, "ioremap failed\n");
+       if (IS_ERR(base))
                return PTR_ERR(base);
-       }
 
        ret = devm_clk_bulk_get(dev, init_data->num_parent_clks,
                                init_data->parent_clks);