sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nb/intel/x4x: Make raminit less verbose with CONFIG_DEBUG_RAM_SETUP

Browse Source 
Hides JEDEC steps using the RAM_SPEW macro.

Also hides a hexdump of SPDs.

Change-Id: Ie2b484cf1f1d296823df0473e852d9d07ca20246
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/18924
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
This commit is contained in:
Arthur Heymans
2017-03-20 16:32:07 +01:00
committed by Nico Huber
parent 3db82be764
commit cfa2eaa4cc
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/northbridge/intel/x4x/raminit.c
View File

@ -55,9 +55,12 @@ static void sdram_read_spds(struct sysinfo *s)
if (j == 62) if (j == 62)
s->dimms[i].card_type = ((u8) status) & 0x1f; s->dimms[i].card_type = ((u8) status) & 0x1f;
} }
if (status >= 0)
if (status >= 0) {
if (IS_ENABLED(CONFIG_DEBUG_RAM_SETUP))
hexdump(s->dimms[i].spd_data, 64); hexdump(s->dimms[i].spd_data, 64);
} }
}
s->spd_type = 0; s->spd_type = 0;
int fail = 1; int fail = 1;

2
src/northbridge/intel/x4x/raminit_ddr2.c
View File

@ -1185,7 +1185,7 @@ static void jedec_ddr2(struct sysinfo *s)
} }
dojedec_ddr2(r + ch*4, ch, jedec[i][0], v); dojedec_ddr2(r + ch*4, ch, jedec[i][0], v);
udelay(1); udelay(1);
//printk(BIOS_DEBUG, "Jedec step %d\n", i); printk(RAM_SPEW, "Jedec step %d\n", i);
} }
} }
printk(BIOS_DEBUG, "MRS done\n"); printk(BIOS_DEBUG, "MRS done\n");