| Message ID | 20230310144734.1546726-1-robh@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | b7405e3f62d68be4cc5f4677499e1b8c13a0d216 |
| Headers | show |
| Series | PM / devfreq: exynos: Use of_property_present() for testing DT property presence | expand |
On 23. 3. 10. 23:47, Rob Herring wrote: > 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> > --- > drivers/devfreq/exynos-bus.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/devfreq/exynos-bus.c b/drivers/devfreq/exynos-bus.c > index 027e8f336acc..d341876e7b62 100644 > --- a/drivers/devfreq/exynos-bus.c > +++ b/drivers/devfreq/exynos-bus.c > @@ -432,7 +432,7 @@ static int exynos_bus_probe(struct platform_device *pdev) > goto err; > > /* Create child platform device for the interconnect provider */ > - if (of_get_property(dev->of_node, "#interconnect-cells", NULL)) { > + if (of_property_present(dev->of_node, "#interconnect-cells")) { > bus->icc_pdev = platform_device_register_data( > dev, "exynos-generic-icc", > PLATFORM_DEVID_AUTO, NULL, 0); Applied it. Thanks.
diff --git a/drivers/devfreq/exynos-bus.c b/drivers/devfreq/exynos-bus.c index 027e8f336acc..d341876e7b62 100644 --- a/drivers/devfreq/exynos-bus.c +++ b/drivers/devfreq/exynos-bus.c @@ -432,7 +432,7 @@ static int exynos_bus_probe(struct platform_device *pdev) goto err; /* Create child platform device for the interconnect provider */ - if (of_get_property(dev->of_node, "#interconnect-cells", NULL)) { + if (of_property_present(dev->of_node, "#interconnect-cells")) { bus->icc_pdev = platform_device_register_data( dev, "exynos-generic-icc", PLATFORM_DEVID_AUTO, NULL, 0);
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> --- drivers/devfreq/exynos-bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)