Re: [PATCH v3 1/6] dt-bindings: mfd: add pf1550

Samuel Kayode
2025-05-28T17:48:30Z

On Wed, May 28, 2025 at 08:08:17AM +0200, Krzysztof Kozlowski wrote:
> > + description:
> > + Temperature threshold for thermal regulation of charger in celsius.
>
> But this now makes me wonder whether this should be just part of thermal
> zone and get the threshold from there. I assume this is temperature of
> CHARGER, not the battery. If battery, you have such properties in
> battery.yaml (monitored-batter).
Yes, it is the charger junction temperature.
>
> @Sebastian,
> Are there existing bindings or devices which regulate temperature based
> on thermal-zones in DT?
> > +examples:
> > + - |
> > + battery: battery-cell {
> > + compatible = "simple-battery";
> > + constant-charge-voltage-max-microvolt = <4400000>;
> > + operating-range-celsius = <0 75>;
>
> So this looks like duplicating thermal-regulation property.
Yes, thermal-regulation should suffice.

Thanks,
Sam