Hi Rafael, Thanks for your work. On 2024-07-29 18:11:29 +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > The only existing caller of thermal_zone_get_num_trips(), which is > rcar_gen3_thermal_probe(), uses this function to put the number of > trip points into a kernel log message, but this information is also > available from the thermal sysfs interface. > > For this reason, remove the thermal_zone_get_num_trips() call from > rcar_gen3_thermal_probe() and drop the former altogether. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > --- > > A new version, based on the discussion following the v1: > > https://lore.kernel.org/linux-pm/20240617183949.GO382677@xxxxxxxxxxxx/ > > The Renesas driver change does not depend on the previous patches > in the series. > > --- > drivers/thermal/renesas/rcar_gen3_thermal.c | 6 +----- > drivers/thermal/thermal_trip.c | 6 ------ > include/linux/thermal.h | 1 - > 3 files changed, 1 insertion(+), 12 deletions(-) > > Index: linux-pm/drivers/thermal/renesas/rcar_gen3_thermal.c > =================================================================== > --- linux-pm.orig/drivers/thermal/renesas/rcar_gen3_thermal.c > +++ linux-pm/drivers/thermal/renesas/rcar_gen3_thermal.c > @@ -563,11 +563,7 @@ static int rcar_gen3_thermal_probe(struc > if (ret) > goto error_unregister; > > - ret = thermal_zone_get_num_trips(tsc->zone); > - if (ret < 0) > - goto error_unregister; > - > - dev_info(dev, "Sensor %u: Loaded %d trip points\n", i, ret); > + dev_info(dev, "Sensor %u: Loaded\n", i); > } > > if (!priv->num_tscs) { > Index: linux-pm/drivers/thermal/thermal_trip.c > =================================================================== > --- linux-pm.orig/drivers/thermal/thermal_trip.c > +++ linux-pm/drivers/thermal/thermal_trip.c > @@ -55,12 +55,6 @@ int thermal_zone_for_each_trip(struct th > } > EXPORT_SYMBOL_GPL(thermal_zone_for_each_trip); > > -int thermal_zone_get_num_trips(struct thermal_zone_device *tz) > -{ > - return tz->num_trips; > -} > -EXPORT_SYMBOL_GPL(thermal_zone_get_num_trips); > - > /** > * thermal_zone_set_trips - Computes the next trip points for the driver > * @tz: a pointer to a thermal zone device structure > Index: linux-pm/include/linux/thermal.h > =================================================================== > --- linux-pm.orig/include/linux/thermal.h > +++ linux-pm/include/linux/thermal.h > @@ -210,7 +210,6 @@ int for_each_thermal_trip(struct thermal > int thermal_zone_for_each_trip(struct thermal_zone_device *tz, > int (*cb)(struct thermal_trip *, void *), > void *data); > -int thermal_zone_get_num_trips(struct thermal_zone_device *tz); > void thermal_zone_set_trip_temp(struct thermal_zone_device *tz, > struct thermal_trip *trip, int temp); > > > > -- Kind Regards, Niklas Söderlund
