Re: [PATCH v1 01/16] thermal: core: Introduce .trip_crossed() callback for thermal governors

[Daniel Lezcano]

On 23/04/2024 19:25, Rafael J. Wysocki wrote:
> On Tue, Apr 23, 2024 at 7:14 PM Daniel Lezcano
> <daniel.lezcano@xxxxxxxxxx> wrote:
> > On 10/04/2024 18:10, Rafael J. Wysocki wrote:
> > > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> > >
> > > Introduce a new thermal governor callback called .trip_crossed()
> > > that will be invoked whenever a trip point is crossed by the zone
> > > temperature, either on the way up or on the way down.
> > >
> > > The trip crossing direction information will be passed to it and if
> > > multiple trips are crossed in the same direction during one thermal zone
> > > update, the new callback will be invoked for them in temperature order,
> > > either ascending or descending, depending on the trip crossing
> > > direction.
> > >
> > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> > > ---

[ ... ]

> > > +             if (governor->trip_crossed)
> > > +                     governor->trip_crossed(tz, &td->trip, true);
> >
> > Is it possible to wrap this into a function ? So we keep the calls at
> > the same level in this block
>
> I can send a separate patch for this if you want me to.

Yes, sure


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog