i2c: bcm2835: Simplify with dev_err_probe()

Common pattern of handling deferred probe can be simplified with
dev_err_probe().  Less code and the error value gets printed.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Wolfram Sang <wsa@kernel.org>

diff --git a/drivers/i2c/busses/i2c-bcm2835.c b/drivers/i2c/busses/i2c-bcm2835.c
index 5dc519516292c92cbe0cc2a8a9b57bce7091b029..37443edbf75464083693878983790c93d3457709 100644 (file)
--- a/drivers/i2c/busses/i2c-bcm2835.c
+++ b/drivers/i2c/busses/i2c-bcm2835.c
@@ -421,11 +421,9 @@ static int bcm2835_i2c_probe(struct platform_device *pdev)
                return PTR_ERR(i2c_dev->regs);
 
        mclk = devm_clk_get(&pdev->dev, NULL);
-       if (IS_ERR(mclk)) {
-               if (PTR_ERR(mclk) != -EPROBE_DEFER)
-                       dev_err(&pdev->dev, "Could not get clock\n");
-               return PTR_ERR(mclk);
-       }
+       if (IS_ERR(mclk))
+               return dev_err_probe(&pdev->dev, PTR_ERR(mclk),
+                                    "Could not get clock\n");
 
        i2c_dev->bus_clk = bcm2835_i2c_register_div(&pdev->dev, mclk, i2c_dev);