diff options
| author | Javier Martinez Canillas <javier@osg.samsung.com> | 2016-04-20 10:37:56 -0400 |
|---|---|---|
| committer | Wolfram Sang <wsa@the-dreams.de> | 2016-04-26 00:32:07 +0200 |
| commit | d16415b2627e91a45778e1888bfdd5c6744294bf (patch) | |
| tree | 98725f1f2487d02d0862e0f29f97143b30272a55 | |
| parent | 8f8edd491ac64c3cfe4e8ffff2b7385df5c02860 (diff) | |
| download | linux-d16415b2627e91a45778e1888bfdd5c6744294bf.tar.gz linux-d16415b2627e91a45778e1888bfdd5c6744294bf.tar.bz2 linux-d16415b2627e91a45778e1888bfdd5c6744294bf.zip | |
i2c: s3c2410: Check clk_prepare_enable() return value
The clk_prepare_enable() function can fail so check the return
value and propagate the error in case of a failure.
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
| -rw-r--r-- | drivers/i2c/busses/i2c-s3c2410.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c index c08830549578..38dc1cacfd8b 100644 --- a/drivers/i2c/busses/i2c-s3c2410.c +++ b/drivers/i2c/busses/i2c-s3c2410.c @@ -1158,7 +1158,12 @@ static int s3c24xx_i2c_probe(struct platform_device *pdev) return -EINVAL; /* initialise the i2c controller */ - clk_prepare_enable(i2c->clk); + ret = clk_prepare_enable(i2c->clk); + if (ret) { + dev_err(&pdev->dev, "I2C clock enable failed\n"); + return ret; + } + ret = s3c24xx_i2c_init(i2c); clk_disable(i2c->clk); if (ret != 0) { |