Re: [PATCH 1/2] thermal: intel: int340x: Add performance control for platform temperature control

["Zhang, Rui" <rui.zhang@intel.com>]

On Thu, 2025-06-05 at 10:25 -0700, srinivas pandruvada wrote:
> On Thu, 2025-06-05 at 03:06 +0000, Zhang, Rui wrote:
> > On Wed, 2025-06-04 at 13:35 -0700, Srinivas Pandruvada wrote:
> > > Add additional attribute to control performance of platform
> > > temperature
> > > control feature. Two attributes are added:
> > > 
> > > gain: 0-7 levels, with 0 being most aggressive.
> > > 	7 – graceful, favors performance at the expense of
> > > temperature
> > > 	overshoots
> > > 	0 – aggressive, favors tight regulation over performance
> > 
> > By reading this, I know that setting "pts_0_control/gain" to a larger
> > value means less aggressive ptc control.
> > 
> > But what does "gain" mean here?
> This is the input to the algorithm on how much performance gain is
> allowed to be in acceptable range for trip violation. May be call
> performance levels instead. I am using the same term as in the spec.

So "graceful" and "aggressive" refers to the temperature control, and
"gain" refers to the performance.

Thanks for clarifying.

-rui

> 
> Thanks,
> Srinivas
> 
> > 
> > May be my English problem, I'm trying hard to understand this, but it
> > is
> > still a bit confusing to me.
> > 
> > thanks,
> > rui