diff options
| author |   Rob Herring (Arm) <robh@kernel.org> | 2024-07-31 13:12:43 -0600 |
|---|---|---|
| committer |   Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2024-08-03 12:19:03 +0100 |
| commit | 958000dfa36d9bf9191af7cdab1b53ab60095100 (patch) | |
| tree | cc17b01fe20d4ecbe9de519355d84e8cbc842cf3 | |
| parent | iio: adc: ad7768-1: use device_* to iterate over device child nodes (diff) | |
| download | wireguard-linux-958000dfa36d9bf9191af7cdab1b53ab60095100.tar.xz wireguard-linux-958000dfa36d9bf9191af7cdab1b53ab60095100.zip | |
iio: adc: aspeed: Use of_property_present()
Use of_property_present() to test for property presence rather than
of_find_property(). This is part of a larger effort to remove callers
of of_find_property() and similar functions. of_find_property() leaks
the DT struct property and data pointers which is a problem for
dynamically allocated nodes which may be freed.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>
Link: https://patch.msgid.link/20240731191312.1710417-5-robh@kernel.org
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
| -rw-r--r-- | drivers/iio/adc/aspeed_adc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iio/adc/aspeed_adc.c b/drivers/iio/adc/aspeed_adc.c index 090416c0d622..064cddaf6399 100644 --- a/drivers/iio/adc/aspeed_adc.c +++ b/drivers/iio/adc/aspeed_adc.c @@ -555,8 +555,7 @@ static int aspeed_adc_probe(struct platform_device *pdev) if (ret) return ret; - if (of_find_property(data->dev->of_node, "aspeed,battery-sensing", - NULL)) { + if (of_property_present(data->dev->of_node, "aspeed,battery-sensing")) { if (data->model_data->bat_sense_sup) { data->battery_sensing = 1; if (readl(data->base + ASPEED_REG_ENGINE_CONTROL) & |