On Sun, 03 Dec 2023, Andreas Kemnade wrote: > Instead of only accepting the ti specific properties accept also > the standard property. For uniformity, search in the parent node Search 'in' the parent node or 'from' the parent node? Where is the property? > for the tag. The code for powering of is also isolated from the Should this be "off"? > rest in this file. So it is a pure Linux design decision to put it > here. > > Signed-off-by: Andreas Kemnade <andreas@xxxxxxxxxxxx> > --- > drivers/mfd/twl4030-power.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/mfd/twl4030-power.c b/drivers/mfd/twl4030-power.c > index e35b0f788c504..3ef892e63b88f 100644 > --- a/drivers/mfd/twl4030-power.c > +++ b/drivers/mfd/twl4030-power.c > @@ -686,6 +686,9 @@ static bool twl4030_power_use_poweroff(const struct twl4030_power_data *pdata, > if (of_property_read_bool(node, "ti,use_poweroff")) > return true; > > + if (of_device_is_system_power_controller(node->parent)) > + return true; > + > return false; > } > > -- > 2.39.2 > -- Lee Jones [李琼斯]
