ARM: OMAP2+: Fix an IS_ERR() vs NULL check in _get_pwrdm()
authorJing Xiangfeng <jingxiangfeng@huawei.com>
Fri, 24 Jul 2020 03:54:30 +0000 (11:54 +0800)
committerTony Lindgren <tony@atomide.com>
Mon, 17 Aug 2020 11:25:23 +0000 (14:25 +0300)
The of_clk_get() function returns error pointers, it never returns NULL.

Fixes: 4ea3711aece4 ("ARM: OMAP2+: omap-iommu.c conversion to ti-sysc")
Signed-off-by: Jing Xiangfeng <jingxiangfeng@huawei.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
arch/arm/mach-omap2/omap-iommu.c

index 54aff33..bfa5e1b 100644 (file)
@@ -74,7 +74,7 @@ static struct powerdomain *_get_pwrdm(struct device *dev)
                return pwrdm;
 
        clk = of_clk_get(dev->of_node->parent, 0);
-       if (!clk) {
+       if (IS_ERR(clk)) {
                dev_err(dev, "no fck found\n");
                return NULL;
        }