diff options
author | Arvind Yadav <arvind.yadav.cs@gmail.com> | 2017-05-09 16:00:28 +0530 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2017-05-16 11:23:28 -0400 |
commit | 5dc63fdcc09f47fb226b8bc7d83a61feb787d817 (patch) | |
tree | 925e21f87e7a046a155a3037abafbec13c9f3b75 /drivers/ata | |
parent | 8bfd174312629866efa535193d9e563768ff4307 (diff) | |
download | linux-5dc63fdcc09f47fb226b8bc7d83a61feb787d817.tar.gz linux-5dc63fdcc09f47fb226b8bc7d83a61feb787d817.tar.bz2 linux-5dc63fdcc09f47fb226b8bc7d83a61feb787d817.zip |
ata: sata_rcar: Handle return value of clk_prepare_enable
Here, Clock enable can failed. So adding an error check for
clk_prepare_enable.
tj: minor style updates
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers/ata')
-rw-r--r-- | drivers/ata/sata_rcar.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c index 5d38245a7a73..b7939a2c1fab 100644 --- a/drivers/ata/sata_rcar.c +++ b/drivers/ata/sata_rcar.c @@ -890,7 +890,10 @@ static int sata_rcar_probe(struct platform_device *pdev) dev_err(&pdev->dev, "failed to get access to sata clock\n"); return PTR_ERR(priv->clk); } - clk_prepare_enable(priv->clk); + + ret = clk_prepare_enable(priv->clk); + if (ret) + return ret; host = ata_host_alloc(&pdev->dev, 1); if (!host) { @@ -970,8 +973,11 @@ static int sata_rcar_resume(struct device *dev) struct ata_host *host = dev_get_drvdata(dev); struct sata_rcar_priv *priv = host->private_data; void __iomem *base = priv->base; + int ret; - clk_prepare_enable(priv->clk); + ret = clk_prepare_enable(priv->clk); + if (ret) + return ret; /* ack and mask */ iowrite32(0, base + SATAINTSTAT_REG); @@ -988,8 +994,11 @@ static int sata_rcar_restore(struct device *dev) { struct ata_host *host = dev_get_drvdata(dev); struct sata_rcar_priv *priv = host->private_data; + int ret; - clk_prepare_enable(priv->clk); + ret = clk_prepare_enable(priv->clk); + if (ret) + return ret; sata_rcar_setup_port(host); |