summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSubrata Banik <subrata.banik@intel.com>2017-08-10 14:28:54 +0530
committerAaron Durbin <adurbin@chromium.org>2017-08-14 17:58:50 +0000
commit4277667499d32e87079cecea1d43c2e612b1535d (patch)
treee534c56f8e4e5d371c92f8190099d08821c9a0e2
parentcdea598e8912e0320bd7337c1b6736025676b429 (diff)
soc/intel/cannonlake: Initialize struct member to 0
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>
-rw-r--r--src/soc/intel/cannonlake/reset.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/soc/intel/cannonlake/reset.c b/src/soc/intel/cannonlake/reset.c
index 93714d3b9a..918c8c7bf9 100644
--- a/src/soc/intel/cannonlake/reset.c
+++ b/src/soc/intel/cannonlake/reset.c
@@ -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) {