diff options
| author | Subrata Banik <subrata.banik@intel.com> | 2017-08-10 14:28:54 +0530 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2017-08-14 17:58:50 +0000 |
| commit | 4277667499d32e87079cecea1d43c2e612b1535d (patch) | |
| tree | e534c56f8e4e5d371c92f8190099d08821c9a0e2 | |
| parent | cdea598e8912e0320bd7337c1b6736025676b429 (diff) | |
| download | coreboot-4277667499d32e87079cecea1d43c2e612b1535d.tar.xz | |
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.c | 2 |
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) { |