Hi, On Fri, Mar 10, 2023 at 08:47:35AM -0600, Rob Herring wrote: > It is preferred to use typed property access functions (i.e. > of_property_read_ functions) rather than low-level > of_get_property/of_find_property functions for reading properties. > Convert reading boolean properties to to of_property_read_bool(). > > Signed-off-by: Rob Herring > --- Thanks, queued. -- Sebastian > drivers/power/supply/charger-manager.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/power/supply/charger-manager.c b/drivers/power/supply/charger-manager.c > index c9e8450c646f..5fa6ba7f41e1 100644 > --- a/drivers/power/supply/charger-manager.c > +++ b/drivers/power/supply/charger-manager.c > @@ -1331,7 +1331,7 @@ static struct charger_desc *of_cm_parse_desc(struct device *dev) > of_property_read_string(np, "cm-thermal-zone", &desc->thermal_zone); > > of_property_read_u32(np, "cm-battery-cold", &desc->temp_min); > - if (of_get_property(np, "cm-battery-cold-in-minus", NULL)) > + if (of_property_read_bool(np, "cm-battery-cold-in-minus")) > desc->temp_min *= -1; > of_property_read_u32(np, "cm-battery-hot", &desc->temp_max); > of_property_read_u32(np, "cm-battery-temp-diff", &desc->temp_diff); > -- > 2.39.2 >