clk: mediatek: clk-mt8173-apmixedsys: Fix iomap not released issue
authorAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Thu, 15 Jun 2023 12:20:51 +0000 (14:20 +0200)
committerStephen Boyd <sboyd@kernel.org>
Fri, 16 Jun 2023 19:20:26 +0000 (12:20 -0700)
In case of error after of_ioremap() the resource must be released:
call iounmap() where appropriate to fix that.

Fixes: 41138fbf876c ("clk: mediatek: mt8173: Migrate to platform driver and common probe")
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20230615122051.546985-4-angelogioacchino.delregno@collabora.com
Reviewed-by: Chen-Yu Tsai <wenst@chromium.org>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/mediatek/clk-mt8173-apmixedsys.c

index c7adcfc..307c24a 100644 (file)
@@ -151,8 +151,10 @@ static int clk_mt8173_apmixed_probe(struct platform_device *pdev)
                return -ENOMEM;
 
        clk_data = mtk_alloc_clk_data(CLK_APMIXED_NR_CLK);
-       if (IS_ERR_OR_NULL(clk_data))
+       if (IS_ERR_OR_NULL(clk_data)) {
+               iounmap(base);
                return -ENOMEM;
+       }
 
        fhctl_parse_dt(fhctl_node, pllfhs, ARRAY_SIZE(pllfhs));
        r = mtk_clk_register_pllfhs(node, plls, ARRAY_SIZE(plls),
@@ -186,6 +188,7 @@ unregister_plls:
                                  ARRAY_SIZE(pllfhs), clk_data);
 free_clk_data:
        mtk_free_clk_data(clk_data);
+       iounmap(base);
        return r;
 }