From: Rob Herring Date: Thu, 9 Feb 2023 23:20:43 +0000 (-0600) Subject: of: Use of_property_present() helper X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=2f0cb4753dd2b5fe71d04407213e2ef253dcdd32;p=linux.git of: Use of_property_present() helper Use of_property_present() instead of of_get_property/of_find_property() in places where we just need to test presence of a property. Reviewed-by: Frank Rowand Tested-by: Frank Rowand Link: https://lore.kernel.org/all/20230215215547.691573-2-robh@kernel.org/ Signed-off-by: Rob Herring --- diff --git a/drivers/of/platform.c b/drivers/of/platform.c index 81c8c227ab6bf..f7d796b540392 100644 --- a/drivers/of/platform.c +++ b/drivers/of/platform.c @@ -528,7 +528,7 @@ static int __init of_platform_default_populate_init(void) int ret; /* Check if we have a MacOS display without a node spec */ - if (of_get_property(of_chosen, "linux,bootx-noscreen", NULL)) { + if (of_property_present(of_chosen, "linux,bootx-noscreen")) { /* * The old code tried to work out which node was the MacOS * display based on the address. I'm dropping that since the diff --git a/drivers/of/property.c b/drivers/of/property.c index 134cfc980b70b..ff71d2ac26cb7 100644 --- a/drivers/of/property.c +++ b/drivers/of/property.c @@ -1086,7 +1086,7 @@ static struct device_node *of_get_compat_node(struct device_node *np) np = NULL; } - if (of_find_property(np, "compatible", NULL)) + if (of_property_present(np, "compatible")) break; np = of_get_next_parent(np);