diff options
author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-05-25 06:30:35 +1000 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2014-05-25 07:53:51 +0200 |
commit | 636cd6134639bff67dc7eb25e3279d588dd7742a (patch) | |
tree | 16ee05f2481f71b2fd39a22b4d085a40498d13dc | |
parent | 2f237c1859f29eaac6b10013a635a2d02a97780e (diff) |
northbridge/intel/i3100/raminit.c: Uninitialized variable
Spotted by Clang
Change-Id: If524a5cd984602a332c4ca28a8167a3597206b94
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/5844
Tested-by: build bot (Jenkins)
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
-rw-r--r-- | src/northbridge/intel/i3100/raminit.c | 2 | ||||
-rw-r--r-- | src/northbridge/intel/i3100/raminit_ep80579.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/northbridge/intel/i3100/raminit.c b/src/northbridge/intel/i3100/raminit.c index e83feaa414..443972e4d6 100644 --- a/src/northbridge/intel/i3100/raminit.c +++ b/src/northbridge/intel/i3100/raminit.c @@ -634,7 +634,7 @@ static void set_on_dimm_termination_enable(const struct mem_controller *ctrl) { u8 c1,c2; u32 dimm,i; - u32 data32; + u32 data32 = 0; u32 t4; /* Set up northbridge values */ diff --git a/src/northbridge/intel/i3100/raminit_ep80579.c b/src/northbridge/intel/i3100/raminit_ep80579.c index 682b1fcb20..90c1005b4c 100644 --- a/src/northbridge/intel/i3100/raminit_ep80579.c +++ b/src/northbridge/intel/i3100/raminit_ep80579.c @@ -503,8 +503,8 @@ static void sdram_set_spd_registers(const struct mem_controller *ctrl) static void set_on_dimm_termination_enable(const struct mem_controller *ctrl) { u8 c1,c2; - u32 dimm,i; - u32 data32; + u32 dimm, i; + u32 data32 = 0; u32 t4; /* Set up northbridge values */ |