Mark Brown, 2024-05-06T00:15:01+09:00: > On Sat, May 04, 2024 at 09:37:06PM +0200, Karel Balej wrote: > > > +static const struct regulator_ops pm886_ldo_ops = { > > + .list_voltage = regulator_list_voltage_table, > > + .map_voltage = regulator_map_voltage_iterate, > > + .set_voltage_sel = regulator_set_voltage_sel_regmap, > > + .get_voltage_sel = regulator_get_voltage_sel_regmap, > > + .enable = regulator_enable_regmap, > > + .disable = regulator_disable_regmap, > > + .is_enabled = regulator_is_enabled_regmap, > > + .get_current_limit = pm886_regulator_get_ilim, > > Do these regulators actually enforce this limit or is this just a spec > limit beyond which regulation may fail? If it's just a spec limit I'd > not expect this operation to be provided, it's more for a hard limit > where the regulator will detect and act on issues. I don't see an error > interrupt or anything and this would be an unusual feature for a LDO. I'm afraid I don't have the answer -- my only reference is the vendor version of the driver and I don't see anything there based on which I would be able to tell. But based on what you write, my guess would be that it's just a spec limit. Should I then drop this op and the max_uA values from all the regulators? Thank you, K. B.