misc/sram: Use of_property_read_bool() for boolean properties
authorRob Herring <robh@kernel.org>
Fri, 10 Mar 2023 14:47:12 +0000 (08:47 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 17 Mar 2023 14:23:25 +0000 (15:23 +0100)
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.
Convert reading boolean properties to to of_property_read_bool().

Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20230310144713.1543544-1-robh@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/sram.c

index dec92580a42c9979fb563ec1e8619375db675a5b..99413310956bbee8a86fbc7396c7b0fff9451a07 100644 (file)
@@ -218,14 +218,9 @@ static int sram_reserve_regions(struct sram_dev *sram, struct resource *res)
                block->res = child_res;
                list_add_tail(&block->list, &reserve_list);
 
-               if (of_find_property(child, "export", NULL))
-                       block->export = true;
-
-               if (of_find_property(child, "pool", NULL))
-                       block->pool = true;
-
-               if (of_find_property(child, "protect-exec", NULL))
-                       block->protect_exec = true;
+               block->export = of_property_read_bool(child, "export");
+               block->pool = of_property_read_bool(child, "pool");
+               block->protect_exec = of_property_read_bool(child, "protect-exec");
 
                if ((block->export || block->pool || block->protect_exec) &&
                    block->size) {