GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nb/intel/i945/raminit: Use 'size_t' for banksize[] 

Change-Id: I4fb845bb4145d47aea39d7e5493d854d00e289aa
Signed-off-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/68213
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Elyes Haouas 2022-10-07 11:25:25 +02:00 committed by Martin L Roth
parent f690c93f25
commit 26fc2a40ae
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/northbridge/intel/i945/raminit.c
View File

@ -426,11 +426,12 @@ static void gather_common_timing(struct sys_info *sysinfo, struct timings *saved
die("DDR-II rank size smaller than 128MB is not supported.\n"); die("DDR-II rank size smaller than 128MB is not supported.\n");
sysinfo->banksize[i * 2] = dimm_info.ranksize_mb / 32; sysinfo->banksize[i * 2] = dimm_info.ranksize_mb / 32;
printk(BIOS_DEBUG, "DIMM %d side 0 = %d MB\n", i, sysinfo->banksize[i * 2] * 32); printk(BIOS_DEBUG, "DIMM %d side 0 = %zu MB\n", i,
sysinfo->banksize[i * 2] * 32);
if (dimm_info.ranks == 2) { if (dimm_info.ranks == 2) {
sysinfo->banksize[(i * 2) + 1] = sysinfo->banksize[(i * 2) + 1] =
dimm_info.ranksize_mb / 32; dimm_info.ranksize_mb / 32;
printk(BIOS_DEBUG, "DIMM %d side 1 = %d MB\n", printk(BIOS_DEBUG, "DIMM %d side 1 = %zu MB\n",
i, sysinfo->banksize[(i * 2) + 1] * 32); i, sysinfo->banksize[(i * 2) + 1] * 32);
} }
@ -1142,7 +1143,7 @@ static void sdram_enable_system_memory_io(struct sys_info *sysinfo)
static int sdram_program_row_boundaries(struct sys_info *sysinfo) static int sdram_program_row_boundaries(struct sys_info *sysinfo)
{ {
int i; int i;
int cum0, cum1, tolud, tom, pci_mmio_size; size_t cum0, cum1, tolud, tom, pci_mmio_size;
const struct device *dev; const struct device *dev;
const struct northbridge_intel_i945_config *cfg = NULL; const struct northbridge_intel_i945_config *cfg = NULL;

2
src/northbridge/intel/i945/raminit.h
View File

@ -49,7 +49,7 @@ struct sys_info {
#define SYSINFO_DIMM_NOT_POPULATED 0x04 #define SYSINFO_DIMM_NOT_POPULATED 0x04
u8 banks[2 * DIMM_SOCKETS]; u8 banks[2 * DIMM_SOCKETS];
u8 banksize[2 * 2 * DIMM_SOCKETS]; size_t banksize[2 * 2 * DIMM_SOCKETS];
const u8 *spd_addresses; const u8 *spd_addresses;
} __packed; } __packed;