diff options
| author | Jiawen Liu <1298662399@qq.com> | 2026-06-18 20:45:16 +0400 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-06-30 12:34:25 +0100 |
| commit | bbbc1ef5b35341c5122418a14f8924688a4bb954 (patch) | |
| tree | f4d91d206f2798cfd0a59ff59a9bd23a55eb7bdd | |
| parent | 7a0d22e632a6f44a40bbe357e081e7a8e806a079 (diff) | |
| download | linux-next-bbbc1ef5b35341c5122418a14f8924688a4bb954.tar.gz linux-next-bbbc1ef5b35341c5122418a14f8924688a4bb954.zip | |
spi: bcm63xx: return error from failed controller suspend
spi_controller_suspend() can fail if the SPI core cannot stop the
controller. bcm63xx_spi_suspend() ignored that error, disabled the
controller clock, and returned success.
Return the suspend error before tearing down the clock.
Signed-off-by: Jiawen Liu <1298662399@qq.com>
Link: https://patch.msgid.link/tencent_0BD7D4091B90EC17A8B2BA5EBA8803725905@qq.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/spi/spi-bcm63xx.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/spi/spi-bcm63xx.c b/drivers/spi/spi-bcm63xx.c index 43d7b54e3ae8..8cef725aab26 100644 --- a/drivers/spi/spi-bcm63xx.c +++ b/drivers/spi/spi-bcm63xx.c @@ -632,8 +632,11 @@ static int bcm63xx_spi_suspend(struct device *dev) { struct spi_controller *host = dev_get_drvdata(dev); struct bcm63xx_spi *bs = spi_controller_get_devdata(host); + int ret; - spi_controller_suspend(host); + ret = spi_controller_suspend(host); + if (ret) + return ret; clk_disable_unprepare(bs->clk); |