erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

soc/intel/cannonlake: Initialize struct member to 0

Browse source 
As per GCC 7.1 compiler struct reset_reply is considered
as uninitialized inside send_heci_reset_message function.

Change-Id: I01b95d31bfb1d2e9af1704a28dacb9cfd1cdcb50
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/20941
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
This commit is contained in:
Subrata Banik 2017-08-10 14:28:54 +05:30 • committed by Aaron Durbin
commit 4277667499
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/soc/intel/cannonlake/reset.c
View file

@ -17,6 +17,7 @@
#include <intelblocks/cse.h>
#include <fsp/util.h>
#include <reset.h>
#include <string.h>
#include <timer.h>
/* Reset Request */
@ -65,6 +66,7 @@ static int send_heci_reset_message(void)
return -1;
reply_size = sizeof(reply);
memset(&reply, 0, reply_size);
heci_receive(&reply, &reply_size);
/* get reply result from HECI MSG */
if (reply.result != 0) {