dmaengine: sh: shdmac: don't open code of_device_get_match_data()

This change will also make Coverity happy by avoiding a theoretical NULL
pointer dereference; yet another reason is to use the above helper function
to tighten the code and make it more readable.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>

diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma/sh/shdmac.c
index 11707df..80d8640 100644 (file)
--- a/drivers/dma/sh/shdmac.c
+++ b/drivers/dma/sh/shdmac.c
@@ -699,7 +699,7 @@ static int sh_dmae_probe(struct platform_device *pdev)
        struct resource *chan, *dmars, *errirq_res, *chanirq_res;
 
        if (pdev->dev.of_node)
-               pdata = of_match_device(sh_dmae_of_match, &pdev->dev)->data;
+               pdata = of_device_get_match_data(&pdev->dev);
        else
                pdata = dev_get_platdata(&pdev->dev);