soc: ti: Use of_property_present() for testing DT property presence
authorRob Herring <robh@kernel.org>
Fri, 10 Mar 2023 14:47:25 +0000 (08:47 -0600)
committerNishanth Menon <nm@ti.com>
Sat, 11 Mar 2023 11:56:54 +0000 (05:56 -0600)
It is preferred to use typed property access functions (i.e.
of_property_read_<type> functions) rather than low-level
of_get_property/of_find_property functions for reading properties. As
part of this, convert of_get_property/of_find_property calls to the
recently added of_property_present() helper when we just want to test
for presence of a property and nothing more.

Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
Link: https://lore.kernel.org/r/20230310144725.1545384-1-robh@kernel.org
drivers/soc/ti/knav_qmss_queue.c
drivers/soc/ti/omap_prm.c

index 8fb7690..48de985 100644 (file)
@@ -1267,7 +1267,7 @@ static int knav_setup_queue_range(struct knav_device *kdev,
        if (of_get_property(node, "qalloc-by-id", NULL))
                range->flags |= RANGE_RESERVED;
 
-       if (of_get_property(node, "accumulator", NULL)) {
+       if (of_property_present(node, "accumulator")) {
                ret = knav_init_acc_range(kdev, node, range);
                if (ret < 0) {
                        devm_kfree(dev, range);
index 913b964..ecd9a8b 100644 (file)
@@ -684,7 +684,7 @@ static int omap_prm_domain_init(struct device *dev, struct omap_prm *prm)
        const char *name;
        int error;
 
-       if (!of_find_property(dev->of_node, "#power-domain-cells", NULL))
+       if (!of_property_present(dev->of_node, "#power-domain-cells"))
                return 0;
 
        of_node_put(dev->of_node);