Re: [PATCH v1 5/6] thermal: gov_step_wise: Fold update_passive_instance() into its caller

From: Rafael J. Wysocki
Date: Fri Oct 20 2023 - 12:31:42 EST


On Fri, Oct 20, 2023 at 6:16 PM Lukasz Luba <lukasz.luba@xxxxxxx> wrote:
>
>
>
> On 10/6/23 18:49, Rafael J. Wysocki wrote:
> > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> >
> > Fold update_passive_instance() into thermal_zone_trip_update() that is
> > its only caller so as to make the code in question easeir to follow.
>
> s/easeir/easier/

I'll fix this in the tree.

> >
> > No intentional functional impact.
> >
> > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> > ---
> > drivers/thermal/gov_step_wise.c | 28 ++++++++++------------------
> > 1 file changed, 10 insertions(+), 18 deletions(-)
> >
>
> other than that LGTM:
>
> Reviewed-by: Lukasz Luba <lukasz.luba@xxxxxxx>

Thank you!