> -----Original Message-----
> From: Markus Theil <markus.th...@tu-ilmenau.de>


> +#include <ctype.h>

This is not needed right.

>  #include <stdio.h>
>  #include <stdlib.h>
>  #include <fcntl.h>
> @@ -96,7 +97,7 @@ power_read_turbo_pct(uint64_t *outVal)
> 
>       errno = 0;
>       *outVal = (uint64_t) strtol(val, &endptr, 10);
> -     if (*endptr != 0 || errno != 0) {
> +     if (errno != 0 || (*endptr != 0 && *endptr != '\n')) {

I encountered today that power library initialization failed and the reason is 
this \n check. 
This fix fixes the issue.

So if you are sending the next version be removing the above mentioned header 
file, please keep my Review and Ack tags in next version.

Reviewed-by: Reshma Pattan <reshma.pat...@intel.com>
Acked-by: Reshma Pattan <reshma.pat...@intel.com>


Reply via email to