diff options
| author | Rob Herring (Arm) <robh@kernel.org> | 2024-07-31 13:13:02 -0600 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2024-08-01 12:50:11 +0100 |
| commit | b03695d95213121719f05cbf6f3baa1ad3667e7f (patch) | |
| tree | eda5e1fcfd7315122e74caf06a5b66adc0c24ecf /drivers/regulator/act8865-regulator.c | |
| parent | cd7d47c5762997a2b1de57a47cd48a1cdad38d96 (diff) | |
| download | lwn-b03695d95213121719f05cbf6f3baa1ad3667e7f.tar.gz lwn-b03695d95213121719f05cbf6f3baa1ad3667e7f.zip | |
regulator: Use of_property_read_bool()
Use of_property_read_bool() to read boolean properties rather than
of_get_property(). This is part of a larger effort to remove callers
of of_get_property() and similar functions. of_get_property() leaks
the DT property data pointer which is a problem for dynamically
allocated nodes which may be freed.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Link: https://patch.msgid.link/20240731191312.1710417-24-robh@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator/act8865-regulator.c')
| -rw-r--r-- | drivers/regulator/act8865-regulator.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/regulator/act8865-regulator.c b/drivers/regulator/act8865-regulator.c index a504b01dd99c..0457af23c55a 100644 --- a/drivers/regulator/act8865-regulator.c +++ b/drivers/regulator/act8865-regulator.c @@ -673,9 +673,7 @@ static int act8865_pmic_probe(struct i2c_client *client) type = (unsigned long) id->data; - voltage_select = !!of_get_property(dev->of_node, - "active-semi,vsel-high", - NULL); + voltage_select = of_property_read_bool(dev->of_node, "active-semi,vsel-high"); } else { type = i2c_id->driver_data; pdata = dev_get_platdata(dev); |