Commit 91df7751 authored by Swati Agarwal's avatar Swati Agarwal Committed by Vinod Koul
Browse files

dmaengine: xilinx_dma: Fix devm_platform_ioremap_resource error handling 



Add missing cleanup in devm_platform_ioremap_resource().
When probe fails remove dma channel resources and disable clocks in
accordance with the order of resources allocated .

Signed-off-by: default avatarSwati Agarwal <swati.agarwal@xilinx.com>
Link: https://lore.kernel.org/r/20220817061125.4720-2-swati.agarwal@xilinx.com


Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 568035b0

drivers/dma/xilinx/xilinx_dma.c

+7 −6
Original line number Diff line number Diff line
@@ -3040,9 +3040,10 @@ static int xilinx_dma_probe(struct platform_device *pdev) 


	/* Request and map I/O memory */

	xdev->regs = devm_platform_ioremap_resource(pdev, 0);

	if (IS_ERR(xdev->regs))

		return PTR_ERR(xdev->regs);



	if (IS_ERR(xdev->regs)) {

		err = PTR_ERR(xdev->regs);

		goto disable_clks;

	}

	/* Retrieve the DMA engine properties from the device tree */

	xdev->max_buffer_len = GENMASK(XILINX_DMA_MAX_TRANS_LEN_MAX - 1, 0);

	xdev->s2mm_chan_id = xdev->dma_config->max_channels / 2;
@@ -3137,7 +3138,7 @@ static int xilinx_dma_probe(struct platform_device *pdev) 
	for_each_child_of_node(node, child) {

		err = xilinx_dma_child_probe(xdev, child);

		if (err < 0)

			goto disable_clks;

			goto error;

	}



	if (xdev->dma_config->dmatype == XDMA_TYPE_VDMA) {
@@ -3172,12 +3173,12 @@ static int xilinx_dma_probe(struct platform_device *pdev) 


	return 0;



disable_clks:

	xdma_disable_allclks(xdev);

error:

	for (i = 0; i < xdev->dma_config->max_channels; i++)

		if (xdev->chan[i])

			xilinx_dma_chan_remove(xdev->chan[i]);

disable_clks:

	xdma_disable_allclks(xdev);



	return err;

}