spi: pic32: Don't leak DMA channels in probe error path
authorLukas Wunner <lukas@wunner.de>
Sun, 8 Nov 2020 22:41:00 +0000 (23:41 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 12 Nov 2020 15:16:34 +0000 (15:16 +0000)
If the calls to devm_request_irq() or devm_spi_register_master() fail
on probe of the PIC32 SPI driver, the DMA channels requested by
pic32_spi_dma_prep() are erroneously not released.  Plug the leak.

Fixes: 1bcb9f8ceb67 ("spi: spi-pic32: Add PIC32 SPI master driver")
Signed-off-by: Lukas Wunner <lukas@wunner.de>
Cc: <stable@vger.kernel.org> # v4.7+
Cc: Purna Chandra Mandal <purna.mandal@microchip.com>
Link: https://lore.kernel.org/r/9624250e3a7aa61274b38219a62375bac1def637.1604874488.git.lukas@wunner.de
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-pic32.c

index 156961b..104bde1 100644 (file)
@@ -839,6 +839,7 @@ static int pic32_spi_probe(struct platform_device *pdev)
        return 0;
 
 err_bailout:
+       pic32_spi_dma_unprep(pic32s);
        clk_disable_unprepare(pic32s->clk);
 err_master:
        spi_master_put(master);