From 097d75398081093f2a95abee0994362ddb838d18 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Mon, 17 Apr 2017 10:14:32 +0200 Subject: nb/intel/pineview/raminit: Fix CONFIG_DEBUG_RAM_SETUP=y not compiling MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The function decode_spd uses undeclared variables and an incorrectly initialized array. Change-Id: Ib45a8b2946c04c270e29524675b1f09d491d282b Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/19336 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Martin Roth --- src/northbridge/intel/pineview/raminit.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/northbridge/intel/pineview') diff --git a/src/northbridge/intel/pineview/raminit.c b/src/northbridge/intel/pineview/raminit.c index e1d23a5679..a45b4da081 100644 --- a/src/northbridge/intel/pineview/raminit.c +++ b/src/northbridge/intel/pineview/raminit.c @@ -109,7 +109,7 @@ static inline int spd_read_byte(unsigned device, unsigned address) return smbus_read_byte(device, address); } -static int decode_spd(struct dimminfo *d) +static int decode_spd(struct dimminfo *d, int i) { d->type = 0; if (d->spd_data[20] == 0x2) { @@ -136,7 +136,7 @@ static int decode_spd(struct dimminfo *d) d->tWR = d->spd_data[36]; d->ranks = d->sides; // XXX #if CONFIG_DEBUG_RAM_SETUP - const char *ubso[] = { {"UB"}, {"SO"} }; + const char *ubso[2] = { "UB", "SO" }; #endif PRINTK_DEBUG("%s-DIMM %d\n", &ubso[d->type][0], i); PRINTK_DEBUG(" Sides : %d\n", d->sides); @@ -305,7 +305,7 @@ static void sdram_read_spds(struct sysinfo *s) int err = 1; FOR_EACH_POPULATED_DIMM(s->dimms, i) { - err = decode_spd(&s->dimms[i]); + err = decode_spd(&s->dimms[i], i); s->dt0mode |= (s->dimms[i].spd_data[49] & 0x2) >> 1; } if (err) { -- cgit v1.2.3