Skip to content

Commit

Permalink
nb/intel/sandybridge: Fix unitialized variable issue
Browse files Browse the repository at this point in the history
Cherry-picked from main branch using commit caa0c0e.

Original commit message:

commit 1e9601c ("nb/intel/sandybridge: Standardize MRC vs. native
SPD mapping API") introduced an uninitialized variable issue.

Change-Id: I41b081dc4c961acc04423067e29e0eabe5f17539
Found-by: Coverity CID 1524317
Original-Signed-off-by: Jeremy Compostella <jeremy.compostella@intel.com>
Original-Reviewed-on: https://review.coreboot.org/c/coreboot/+/79093
Original-Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Original-Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Original-Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Signed-off-by: Felix Singer <felixsinger@posteo.net>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79231
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin L Roth <gaumless@gmail.com>
  • Loading branch information
jeremy-compostella authored and martinlroth committed Nov 23, 2023
1 parent c8dc4a3 commit 2ff2409
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/northbridge/intel/sandybridge/raminit_mrc.c
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ static void devicetree_fill_pei_data(struct pei_data *pei_data)
static void spd_fill_pei_data(struct pei_data *pei_data)
{
struct spd_info spdi = {0};
unsigned int i, have_memory_down;
unsigned int i, have_memory_down = 0;

mb_get_spd_map(&spdi);

Expand Down

0 comments on commit 2ff2409

Please sign in to comment.