]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
of/platform: Use typed accessors rather than of_get_property()
authorRob Herring (Arm) <robh@kernel.org>
Wed, 12 Mar 2025 21:29:36 +0000 (16:29 -0500)
committerRob Herring (Arm) <robh@kernel.org>
Fri, 14 Mar 2025 23:01:42 +0000 (18:01 -0500)
Use the typed of_property_* functions rather than of_get_property()
which leaks pointers to DT data without any control of the lifetime.

Signed-off-by: "Rob Herring (Arm)" <robh@kernel.org>
Link: https://lore.kernel.org/r/20250312212937.1067088-1-robh@kernel.org
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
drivers/of/platform.c

index 242172e4b8757eec9a7ccb413764b475046dbae8..f77cb19973a5d1e1fd47e28848f2d1ec10dd3c52 100644 (file)
@@ -536,8 +536,8 @@ static int __init of_platform_default_populate_init(void)
                 * ignore errors for the rest.
                 */
                for_each_node_by_type(node, "display") {
-                       if (!of_get_property(node, "linux,opened", NULL) ||
-                           !of_get_property(node, "linux,boot-display", NULL))
+                       if (!of_property_read_bool(node, "linux,opened") ||
+                           !of_property_read_bool(node, "linux,boot-display"))
                                continue;
                        dev = of_platform_device_create(node, "of-display", NULL);
                        of_node_put(node);
@@ -551,7 +551,7 @@ static int __init of_platform_default_populate_init(void)
                        char buf[14];
                        const char *of_display_format = "of-display.%d";
 
-                       if (!of_get_property(node, "linux,opened", NULL) || node == boot_display)
+                       if (!of_property_read_bool(node, "linux,opened") || node == boot_display)
                                continue;
                        ret = snprintf(buf, sizeof(buf), of_display_format, display_number++);
                        if (ret < sizeof(buf))