]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
spi: geni-qcom: Drop unused msg parameter from timeout handlers
authorPraveen Talari <praveen.talari@oss.qualcomm.com>
Wed, 4 Feb 2026 16:28:53 +0000 (21:58 +0530)
committerMark Brown <broonie@kernel.org>
Wed, 4 Feb 2026 17:37:35 +0000 (17:37 +0000)
The timeout helpers (handle_se_timeout() and handle_gpi_timeout())
took a 'struct spi_message *msg' that was never used. Remove this
parameter and adjust the callers to simplify the interfaces.

Signed-off-by: Praveen Talari <praveen.talari@oss.qualcomm.com>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Link: https://patch.msgid.link/20260204162854.1206323-4-praveen.talari@oss.qualcomm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-geni-qcom.c

index e48aea7ce681a8275a90f424da067f8ac41c208d..5077dc041e3ab4bef9af7b7c1c6efc7f3b033c89 100644 (file)
@@ -146,8 +146,7 @@ static int get_spi_clk_cfg(unsigned int speed_hz,
        return ret;
 }
 
-static void handle_se_timeout(struct spi_controller *spi,
-                             struct spi_message *msg)
+static void handle_se_timeout(struct spi_controller *spi)
 {
        struct spi_geni_master *mas = spi_controller_get_devdata(spi);
        unsigned long time_left;
@@ -222,7 +221,7 @@ reset_if_dma:
        }
 }
 
-static void handle_gpi_timeout(struct spi_controller *spi, struct spi_message *msg)
+static void handle_gpi_timeout(struct spi_controller *spi)
 {
        struct spi_geni_master *mas = spi_controller_get_devdata(spi);
 
@@ -237,10 +236,10 @@ static void spi_geni_handle_err(struct spi_controller *spi, struct spi_message *
        switch (mas->cur_xfer_mode) {
        case GENI_SE_FIFO:
        case GENI_SE_DMA:
-               handle_se_timeout(spi, msg);
+               handle_se_timeout(spi);
                break;
        case GENI_GPI_DMA:
-               handle_gpi_timeout(spi, msg);
+               handle_gpi_timeout(spi);
                break;
        default:
                dev_err(mas->dev, "Abort on Mode:%d not supported", mas->cur_xfer_mode);