Hi Boris On Fri, Jan 06, 2017 at 12:16:17PM +0100, Borislav Petkov wrote: > On Thu, Jan 05, 2017 at 05:26:17PM -0800, Raj, Ashok wrote: > > Agree, since we have both a log and another agent to deal with it, it makes > > no good reason to continue... Will pass this along, and have someone look at > > cleaning this up. > > Like this?
That was quick :-). > - if (therm_throt_process(msr_val & THERM_STATUS_PROCHOT, > - THERMAL_THROTTLING_EVENT, > - CORE_LEVEL) != 0) Looks like we don't need a return value from therm_throt_process(), we can fix that as void as well. Otherwise it looks good. > + therm_throt_process(msr_val & THERM_STATUS_PROCHOT, > + THERMAL_THROTTLING_EVENT, > + CORE_LEVEL); > Cheers, Ashok