]> Gentwo Git Trees - linux/.git/commitdiff
drm/mediatek: Fix probe device leaks
authorJohan Hovold <johan@kernel.org>
Tue, 23 Sep 2025 15:23:38 +0000 (17:23 +0200)
committerChun-Kuang Hu <chunkuang.hu@kernel.org>
Mon, 17 Nov 2025 15:04:42 +0000 (15:04 +0000)
Make sure to drop the reference taken to each component device during
probe on probe failure (e.g. probe deferral) and on driver unbind.

Fixes: 6ea6f8276725 ("drm/mediatek: Use correct device pointer to get CMDQ client register")
Cc: stable@vger.kernel.org # 5.12
Cc: Chun-Kuang Hu <chunkuang.hu@kernel.org>
Signed-off-by: Johan Hovold <johan@kernel.org>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://patchwork.kernel.org/project/dri-devel/patch/20250923152340.18234-4-johan@kernel.org/
Signed-off-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>
drivers/gpu/drm/mediatek/mtk_ddp_comp.c

index 31d67a131c5031c56800b90b6227c8c419f8ff07..9672ea1f91a2b5635ab680feb53151609da5d1ac 100644 (file)
@@ -621,6 +621,13 @@ int mtk_find_possible_crtcs(struct drm_device *drm, struct device *dev)
        return ret;
 }
 
+static void mtk_ddp_comp_put_device(void *_dev)
+{
+       struct device *dev = _dev;
+
+       put_device(dev);
+}
+
 static void mtk_ddp_comp_clk_put(void *_clk)
 {
        struct clk *clk = _clk;
@@ -656,6 +663,10 @@ int mtk_ddp_comp_init(struct device *dev, struct device_node *node, struct mtk_d
        }
        comp->dev = &comp_pdev->dev;
 
+       ret = devm_add_action_or_reset(dev, mtk_ddp_comp_put_device, comp->dev);
+       if (ret)
+               return ret;
+
        if (type == MTK_DISP_AAL ||
            type == MTK_DISP_BLS ||
            type == MTK_DISP_CCORR ||