diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2018-03-22 02:11:48 +0000 |
---|---|---|
committer | Stephen Boyd <sboyd@kernel.org> | 2018-03-23 09:44:45 -0700 |
commit | fc3fcb4ff1960e9f0b72b1d4855f7e0bcf18b7f2 (patch) | |
tree | 5dc121a9b685d43d7a5c8c4efdcda26ed9ccf3d4 /drivers/clk/davinci | |
parent | 58e1e2d2cd89a4aa77212eae64dd4824374e83f4 (diff) | |
download | lwn-fc3fcb4ff1960e9f0b72b1d4855f7e0bcf18b7f2.tar.gz lwn-fc3fcb4ff1960e9f0b72b1d4855f7e0bcf18b7f2.zip |
clk: davinci: Remove redundant dev_err calls
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>
Diffstat (limited to 'drivers/clk/davinci')
-rw-r--r-- | drivers/clk/davinci/pll.c | 4 | ||||
-rw-r--r-- | drivers/clk/davinci/psc.c | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/drivers/clk/davinci/pll.c b/drivers/clk/davinci/pll.c index 89d30bf95102..23a24c944f1d 100644 --- a/drivers/clk/davinci/pll.c +++ b/drivers/clk/davinci/pll.c @@ -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); } diff --git a/drivers/clk/davinci/psc.c b/drivers/clk/davinci/psc.c index 3b0e59dfbdd7..ff6f4a038e04 100644 --- a/drivers/clk/davinci/psc.c +++ b/drivers/clk/davinci/psc.c @@ -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); |