[PATCH kernel 1/1] powerpc/Documentation/misc-devices/: Fix one compile error

Winkler, Tomas tomas.winkler at intel.com
Wed Feb 22 19:14:11 AEDT 2017


On Wed, 2017-02-22 at 15:22 +0800, fgao at ikuai8.com wrote:
> From: Gao Feng <fgao at ikuai8.com>
> 
> When make allyesconfig, there is one compile error on my platform
> "gcc (Ubuntu 4.8.4-2ubuntu1~14.04.3) 4.8.4".
> The following is the output.
> 
> Documentation/misc-devices/mei/mei-amt-version.c: In function ‘main’:
> Documentation/misc-devices/mei/mei-amt-version.c:103:5: warning:
> ‘acmd.fd’
> is used uninitialized in this function [-Wuninitialized]
>   if (cl->fd != -1)
>      ^
> Documentation/misc-devices/mei/mei-amt-version.c:443:21: note:
> ‘acmd.fd’
> was declared here
>   struct amt_host_if acmd;
>                      ^
> This commit fixes this compile error.
> 
> Signed-off-by: Gao Feng <fgao at ikuai8.com>

This is false positive, as the variable is assined in mei_init(), in
any case, the code has moved under samples diretory in the current
kernel. Anything need to be fixed there first

Thanks 
Tomas

> ---
>  Documentation/misc-devices/mei/mei-amt-version.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/misc-devices/mei/mei-amt-version.c
> b/Documentation/misc-devices/mei/mei-amt-version.c
> index 49e4f77..ca035cb 100644
> --- a/Documentation/misc-devices/mei/mei-amt-version.c
> +++ b/Documentation/misc-devices/mei/mei-amt-version.c
> @@ -297,6 +297,7 @@ static bool amt_host_if_init(struct amt_host_if
> *acmd,
>  		      unsigned long send_timeout, bool verbose)
>  {
>  	acmd->send_timeout = (send_timeout) ? send_timeout : 20000;
> +	acmd->mei_cl.fd = -1;
>  	acmd->initialized = mei_init(&acmd->mei_cl, &MEI_IAMTHIF, 0,
> verbose);
>  	return acmd->initialized;
>  }


More information about the Linuxppc-dev mailing list