linux-kernel - [PATCH resubmit] spi: atmel-quadspi: remove references to runtime PM on error path

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20250127094258.841684-2-csokas.bence@prolan.hu>
Date: Mon, 27 Jan 2025 10:42:58 +0100
From: Bence Csókás <csokas.bence@...lan.hu>
To: Tudor Ambarus <tudor.ambarus@...aro.org>, Mark Brown <broonie@...nel.org>,
	Csókás, Bence <csokas.bence@...lan.hu>, "Varshini
 Rajendran" <varshini.rajendran@...rochip.com>, <linux-spi@...r.kernel.org>,
	<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
CC: Claudiu Beznea <claudiu.beznea@...on.dev>, Durai Manickam KR
	<durai.manickamkr@...rochip.com>, Alexander Dahl <ada@...rsis.com>, "Nicolas
 Ferre" <nicolas.ferre@...rochip.com>, Alexandre Belloni
	<alexandre.belloni@...tlin.com>
Subject: [PATCH resubmit] spi: atmel-quadspi: remove references to runtime PM on error path

From: Claudiu Beznea <claudiu.beznea@...rochip.com>

There is no need to call runtime PM put APIs on error path of
`atmel_qspi_sama7g5_transfer()` as the caller (`atmel_qspi_exec_op()`)
of it will take care of this if needed.

Fixes: 5af42209a4d2 ("spi: atmel-quadspi: Add support for sama7g5 QSPI")
Signed-off-by: Claudiu Beznea <claudiu.beznea@...rochip.com>
Signed-off-by: Durai Manickam KR <durai.manickamkr@...rochip.com>
Reported-by: Alexander Dahl <ada@...rsis.com>
Closes: https://lore.kernel.org/linux-spi/20250109-carat-festivity-5f088e1add3c@thorsis.com/
[ csokas.bence: Rebase and clarify msg, fix/add tags ]
Signed-off-by: Bence Csókás <csokas.bence@...lan.hu>
---

Notes:
    Rebased onto current spi-next:
    commit ff9e24437b18 ("Merge remote-tracking branch 'spi/for-6.13' into spi-linus")

 drivers/spi/atmel-quadspi.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/spi/atmel-quadspi.c b/drivers/spi/atmel-quadspi.c
index abdc49d9d940..e2235d71c822 100644
--- a/drivers/spi/atmel-quadspi.c
+++ b/drivers/spi/atmel-quadspi.c
@@ -930,11 +930,8 @@ static int atmel_qspi_sama7g5_transfer(struct spi_mem *mem,
 
 	/* Release the chip-select. */
 	ret = atmel_qspi_reg_sync(aq);
-	if (ret) {
-		pm_runtime_mark_last_busy(&aq->pdev->dev);
-		pm_runtime_put_autosuspend(&aq->pdev->dev);
+	if (ret)
 		return ret;
-	}
 	atmel_qspi_write(QSPI_CR_LASTXFER, aq, QSPI_CR);
 
 	return atmel_qspi_wait_for_completion(aq, QSPI_SR_CSRA);
-- 
2.48.1