Replace resource handling in the driver with managed device resource.

Signed-off-by: Alexander Shiyan <shc_w...@mail.ru>
---
 drivers/media/platform/m2m-deinterlace.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/media/platform/m2m-deinterlace.c 
b/drivers/media/platform/m2m-deinterlace.c
index c21d14f..d36c507 100644
--- a/drivers/media/platform/m2m-deinterlace.c
+++ b/drivers/media/platform/m2m-deinterlace.c
@@ -1002,7 +1002,7 @@ static int deinterlace_probe(struct platform_device *pdev)
        dma_cap_mask_t mask;
        int ret = 0;
 
-       pcdev = kzalloc(sizeof *pcdev, GFP_KERNEL);
+       pcdev = devm_kzalloc(&pdev->dev, sizeof(*pcdev), GFP_KERNEL);
        if (!pcdev)
                return -ENOMEM;
 
@@ -1012,7 +1012,7 @@ static int deinterlace_probe(struct platform_device *pdev)
        dma_cap_set(DMA_INTERLEAVE, mask);
        pcdev->dma_chan = dma_request_channel(mask, NULL, pcdev);
        if (!pcdev->dma_chan)
-               goto free_dev;
+               return -ENODEV;
 
        if (!dma_has_cap(DMA_INTERLEAVE, pcdev->dma_chan->device->cap_mask)) {
                v4l2_err(&pcdev->v4l2_dev, "DMA does not support INTERLEAVE\n");
@@ -1078,8 +1078,6 @@ unreg_dev:
        v4l2_device_unregister(&pcdev->v4l2_dev);
 rel_dma:
        dma_release_channel(pcdev->dma_chan);
-free_dev:
-       kfree(pcdev);
 
        return ret;
 }
@@ -1094,7 +1092,6 @@ static int deinterlace_remove(struct platform_device 
*pdev)
        v4l2_device_unregister(&pcdev->v4l2_dev);
        vb2_dma_contig_cleanup_ctx(pcdev->alloc_ctx);
        dma_release_channel(pcdev->dma_chan);
-       kfree(pcdev);
 
        return 0;
 }
-- 
1.8.5.5

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to