> -----Original Message-----
> From: Markus Theil <[email protected]>


> +#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 <[email protected]>
Acked-by: Reshma Pattan <[email protected]>


Reply via email to