Re: [PATCH v5 3/3] regulator: Add TI LM51772 support

[Mark Brown]

On Tue, Nov 10, 2026 at 09:26:04AM +0100, Paul Louvel wrote:
> On Fri Nov 6, 2026 at 12:17 PM CET, Paul Louvel wrote:

> > Maybe would it be better if the core could also change the active discharge
> > strength:
> >
> >   int (*set_active_discharge)(struct regulator_dev *rdev, bool enable,
> >             int strength_uA);
> >
> > That would remove the need for introducing a vendor-specific property:
> > 'ti,active-discharge-strength-microamp'.

> Let me know what you think.

I'm not sure how general that is, it's very common for configuration to
be done in terms of a resistance rather than some current regulated
thing (which is a lot harder to implement, unless your thing is actually
just a resistor and assuming a voltage or something).

Attachment: signature.asc
Description: PGP signature