@@ -100,6 +100,8 @@ struct zn_coefficients {
s32 base_trough;
s32 oscillation_count;
enum pivot_type prev_pivot;
+
+ int zn_state;
};
/**
@@ -431,7 +433,7 @@ static inline s32 get_oscillation_count(s32 curr_err,
* get_zn_state() - Update and get the current Ziegler-Nichols State
* @tzp: The thermal zone params to check to determine the current state
* @zn_state: The current state which should be returned if no changes are
- * made
+ * made
*
* Return: The next zieger-nichols state for this pass of the PID controller
*/
@@ -514,9 +516,21 @@ static inline void ziegler_nichols(struct thermal_zone_device *tz, s32 next_err,
bool is_safe =
is_temperature_safe((control_temp - next_err), control_temp);
- if (tz->tzp->ziegler_nichols == ZN_RESET) {
+ zn_coeffs->zn_state = get_zn_state(tz->tzp, zn_coeffs->zn_state);
+ switch (zn_coeffs->zn_state) {
+ case ZN_ORIGINAL: {
+ set_original_pid_coefficients(tz->tzp);
+ zn_coeffs->zn_state = ZN_OFF;
+ return;
+ }
+ case ZN_RESET: {
reset_ziegler_nichols(zn_coeffs);
- tz->tzp->ziegler_nichols = ZN_ON;
+ zn_coeffs->zn_state = ZN_ON;
+ break;
+ }
+
+ case ZN_OFF:
+ return;
}
/* Override default PID Coefficients. These will be updated later according to the
@@ -556,7 +570,7 @@ static inline void ziegler_nichols(struct thermal_zone_device *tz, s32 next_err,
} else {
set_zn_pid_coefficients(tz->tzp, zn_coeffs->period,
zn_coeffs->k_ultimate);
- tz->tzp->ziegler_nichols = ZN_OFF;
+ zn_coeffs->zn_state = ZN_OFF;
}
return;
@@ -573,7 +587,7 @@ static inline void ziegler_nichols(struct thermal_zone_device *tz, s32 next_err,
zn_coeffs->k_ultimate);
((struct power_allocator_params *)tz->governor_data)
->err_integral = 0;
- tz->tzp->ziegler_nichols = ZN_OFF;
+ zn_coeffs->zn_state = ZN_OFF;
} else {
if (peak_trough == PEAK)
zn_coeffs->t_prev_peak = t_now;
@@ -613,24 +627,7 @@ static u32 pid_controller(struct thermal_zone_device *tz,
err = control_temp - tz->temperature;
- switch (tz->tzp->ziegler_nichols) {
- case ZN_ORIGINAL: {
- set_original_pid_coefficients(tz->tzp);
- tz->tzp->ziegler_nichols = ZN_OFF;
- break;
- }
- case ZN_RESET: {
- ziegler_nichols(tz, err, control_temp);
- tz->tzp->ziegler_nichols = ZN_ON;
- break;
- }
- case ZN_ON: {
- ziegler_nichols(tz, err, control_temp);
- break;
- }
- default:
- break;
- }
+ ziegler_nichols(tz, err, control_temp);
err = int_to_frac(err);
@@ -1064,6 +1061,7 @@ static int power_allocator_bind(struct thermal_zone_device *tz)
return -ENOMEM;
params->zn_coeffs = zn_coeffs;
+ zn_coeffs->zn_state = ZN_ON;
if (!tz->tzp) {
tz->tzp = kzalloc(sizeof(*tz->tzp), GFP_KERNEL);
@@ -342,7 +342,6 @@ create_s32_tzp_attr(k_po);
create_s32_tzp_attr(k_pu);
create_s32_tzp_attr(k_i);
create_s32_tzp_attr(k_d);
-create_s32_tzp_attr(ziegler_nichols);
create_s32_tzp_attr(integral_cutoff);
create_s32_tzp_attr(slope);
create_s32_tzp_attr(offset);
@@ -376,7 +375,6 @@ static struct attribute *thermal_zone_dev_attrs[] = {
&dev_attr_k_pu.attr,
&dev_attr_k_i.attr,
&dev_attr_k_d.attr,
- &dev_attr_ziegler_nichols.attr,
&dev_attr_integral_cutoff.attr,
&dev_attr_slope.attr,
&dev_attr_offset.attr,
@@ -282,13 +282,6 @@ struct thermal_zone_params {
* Used by thermal zone drivers (default 0).
*/
int offset;
-
- /*
- * Ziegler-Nichols estimation setting. Allows the user to decide
- * whether to use original PID coefficients or calculate using
- * the Ziegler-Nichols algorithm
- */
- s32 ziegler_nichols;
};
/**
The previous patch introduced a sysfs entry to track the Ziegler-Nichols state. This patch will remove it, if it is unwanted. Signed-off-by: Chetankumar Mistry <chetan.mistry@arm.com> --- drivers/thermal/gov_power_allocator.c | 44 +++++++++++++-------------- drivers/thermal/thermal_sysfs.c | 2 -- include/linux/thermal.h | 7 ----- 3 files changed, 21 insertions(+), 32 deletions(-)