memory: renesas-rpc-if: Return error in case devm_ioremap_resource() fails
authorLad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Mon, 25 Oct 2021 20:56:29 +0000 (21:56 +0100)
committerKrzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Tue, 16 Nov 2021 10:28:56 +0000 (11:28 +0100)
Make sure we return error in case devm_ioremap_resource() fails for dirmap
resource.

Fixes: ca7d8b980b67 ("memory: add Renesas RPC-IF driver")
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/20211025205631.21151-6-prabhakar.mahadev-lad.rj@bp.renesas.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
drivers/memory/renesas-rpc-if.c

index 7435baa..ff8bcbc 100644 (file)
@@ -243,7 +243,7 @@ int rpcif_sw_init(struct rpcif *rpc, struct device *dev)
        res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "dirmap");
        rpc->dirmap = devm_ioremap_resource(&pdev->dev, res);
        if (IS_ERR(rpc->dirmap))
-               rpc->dirmap = NULL;
+               return PTR_ERR(rpc->dirmap);
        rpc->size = resource_size(res);
 
        rpc->rstc = devm_reset_control_get_exclusive(&pdev->dev, NULL);