Commit 347de6ba authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Mark Brown

spi/s3c64xx: Do not ignore return value of spi_master_resume/suspend

During PM resume and suspend do not ignore the return value of
spi_master_suspend() or spi_master_resume(). Instead pass it further.
Signed-off-by: default avatarKrzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: default avatarKyungmin Park <kyungmin.park@samsung.com>
Reviewed-by: default avatarSylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: default avatarMark Brown <broonie@linaro.org>
parent ebd805cc
...@@ -1477,7 +1477,9 @@ static int s3c64xx_spi_suspend(struct device *dev) ...@@ -1477,7 +1477,9 @@ static int s3c64xx_spi_suspend(struct device *dev)
struct spi_master *master = dev_get_drvdata(dev); struct spi_master *master = dev_get_drvdata(dev);
struct s3c64xx_spi_driver_data *sdd = spi_master_get_devdata(master); struct s3c64xx_spi_driver_data *sdd = spi_master_get_devdata(master);
spi_master_suspend(master); int ret = spi_master_suspend(master);
if (ret)
return ret;
/* Disable the clock */ /* Disable the clock */
clk_disable_unprepare(sdd->src_clk); clk_disable_unprepare(sdd->src_clk);
...@@ -1503,9 +1505,7 @@ static int s3c64xx_spi_resume(struct device *dev) ...@@ -1503,9 +1505,7 @@ static int s3c64xx_spi_resume(struct device *dev)
s3c64xx_spi_hwinit(sdd, sdd->port_id); s3c64xx_spi_hwinit(sdd, sdd->port_id);
spi_master_resume(master); return spi_master_resume(master);
return 0;
} }
#endif /* CONFIG_PM_SLEEP */ #endif /* CONFIG_PM_SLEEP */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment