Messages in this thread | | | Date | Fri, 6 Jan 2017 07:58:31 -0800 | From | "Raj, Ashok" <> | Subject | Re: Dell XPS13: MCE (Hardware Error) reported |
| |
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
| |