spi: spi-nxp-fspi: use guard(mutex) to simplify the code
authorHaibo Chen <haibo.chen@nxp.com>
Mon, 28 Apr 2025 10:06:45 +0000 (18:06 +0800)
committerMark Brown <broonie@kernel.org>
Tue, 6 May 2025 14:17:29 +0000 (23:17 +0900)
Use guard(mutex) to simplify the code logic.

Signed-off-by: Haibo Chen <haibo.chen@nxp.com>
Link: https://patch.msgid.link/20250428-flexspipatch-v3-3-61d5e8f591bc@nxp.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-nxp-fspi.c

index f7c3330a19009223cbffae8673d264246418e561..ca0b51a254c15f45a9e91907900d94d77f1d5dff 100644 (file)
@@ -932,11 +932,10 @@ static int nxp_fspi_exec_op(struct spi_mem *mem, const struct spi_mem_op *op)
        struct nxp_fspi *f = spi_controller_get_devdata(mem->spi->controller);
        int err = 0;
 
-       mutex_lock(&f->lock);
+       guard(mutex)(&f->lock);
 
        err = pm_runtime_get_sync(f->dev);
        if (err < 0) {
-               mutex_unlock(&f->lock);
                dev_err(f->dev, "Failed to enable clock %d\n", __LINE__);
                return err;
        }
@@ -972,7 +971,6 @@ static int nxp_fspi_exec_op(struct spi_mem *mem, const struct spi_mem_op *op)
        pm_runtime_mark_last_busy(f->dev);
        pm_runtime_put_autosuspend(f->dev);
 
-       mutex_unlock(&f->lock);
        return err;
 }