From: Minghao Chi (CGEL ZTE) <chi.ming...@zte.com.cn>

Use of_device_get_match_data() to simplify the code.
v1->v2:
        Add a judgment on the return value of the A function as NULL

Reported-by: Zeal Robot <zea...@zte.com.cn>
Signed-off-by: Minghao Chi (CGEL ZTE) <chi.ming...@zte.com.cn>
---
 arch/powerpc/platforms/83xx/suspend.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/arch/powerpc/platforms/83xx/suspend.c 
b/arch/powerpc/platforms/83xx/suspend.c
index bb147d34d4a6..6d47a5b81485 100644
--- a/arch/powerpc/platforms/83xx/suspend.c
+++ b/arch/powerpc/platforms/83xx/suspend.c
@@ -322,18 +322,15 @@ static const struct platform_suspend_ops 
mpc83xx_suspend_ops = {
 static const struct of_device_id pmc_match[];
 static int pmc_probe(struct platform_device *ofdev)
 {
-       const struct of_device_id *match;
        struct device_node *np = ofdev->dev.of_node;
        struct resource res;
        const struct pmc_type *type;
        int ret = 0;
 
-       match = of_match_device(pmc_match, &ofdev->dev);
-       if (!match)
+       type = of_device_get_match_data(&ofdev->dev);
+       if (!type)
                return -EINVAL;
 
-       type = match->data;
-
        if (!of_device_is_available(np))
                return -ENODEV;
 
-- 
2.25.1

Reply via email to