diff options
| author |  Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
| 2023-05-30 10:16:48 +0200 |
|---|---|---|
| committer |  Mark Brown <broonie@kernel.org>
| 2023-05-30 13:46:57 +0100 |
| commit | 6f089e986778d3657247fdc2b38bd38de796732b (patch) | |
| tree | bbe36719e7f0c0e970dc0d6cb0c18f5f41730428 /drivers/spi | |
| parent | spi: mt65xx: Convert to platform remove callback returning void (diff) | |
| download | linux-6f089e986778d3657247fdc2b38bd38de796732b.tar.gz linux-6f089e986778d3657247fdc2b38bd38de796732b.tar.bz2 linux-6f089e986778d3657247fdc2b38bd38de796732b.zip | |
spi: mt65xx: Don't disguise a "return 0" as "return ret"
Because of the earlier
if (ret)
return ret;
ret is always zero at the end of mtk_spi_suspend(). Write it as explicit
return 0 for slightly improved clearness.
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20230530081648.2199419-4-u.kleine-koenig@pengutronix.de
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi')
| -rw-r--r-- | drivers/spi/spi-mt65xx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/spi-mt65xx.c b/drivers/spi/spi-mt65xx.c index f532cee3461e..5a0b04c1c755 100644 --- a/drivers/spi/spi-mt65xx.c +++ b/drivers/spi/spi-mt65xx.c @@ -1312,7 +1312,7 @@ static int mtk_spi_suspend(struct device *dev) clk_disable_unprepare(mdata->spi_hclk); } - return ret; + return 0; } static int mtk_spi_resume(struct device *dev) |