diff options
| author | Praveen Talari <praveen.talari@oss.qualcomm.com> | 2026-02-04 21:58:53 +0530 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-02-04 17:37:35 +0000 |
| commit | 9934383f98c91d8dfdf16d0e980431041c2d17ee (patch) | |
| tree | bd4c822bdf209af88b9669a4020a2ef02de43211 | |
| parent | 96e041647bb0f9d92f95df1d69cb7442d7408b79 (diff) | |
spi: geni-qcom: Drop unused msg parameter from timeout handlers
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>
| -rw-r--r-- | drivers/spi/spi-geni-qcom.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/spi/spi-geni-qcom.c b/drivers/spi/spi-geni-qcom.c index e48aea7ce681..5077dc041e3a 100644 --- a/drivers/spi/spi-geni-qcom.c +++ b/drivers/spi/spi-geni-qcom.c @@ -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); |