nb/intel/i945/raminit: Use 'bool' for clkcfg_bit7

Signed-off-by: Elyes Haouas <ehaouas@noos.fr>
Change-Id: Ia87fbbeb9ecb57ee2f4879404cbae5403de9bfc7
Reviewed-on: https://review.coreboot.org/c/coreboot/+/68216
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 12:05:38 +02:00 committed by Martin L Roth
parent 5bbdb0c948
commit d9dade3cb9
2 changed files with 6 additions and 10 deletions

View File

@ -1604,7 +1604,8 @@ static void sdram_program_pll_settings(struct sys_info *sysinfo)
static void sdram_program_graphics_frequency(struct sys_info *sysinfo)
{
u8 reg8;
u8 freq, second_vco, voltage;
u8 freq, voltage;
bool second_vco = false;
#define CRCLK_166MHz 0x00
#define CRCLK_200MHz 0x01
@ -1680,10 +1681,8 @@ static void sdram_program_graphics_frequency(struct sys_info *sysinfo)
else
sysinfo->mvco4x = 0;
second_vco = 0;
if (voltage == VOLTAGE_1_50) {
second_vco = 1;
second_vco = true;
} else if ((i945_silicon_revision() > 0) && (freq == CRCLK_250MHz)) {
u16 mem = sysinfo->memory_frequency;
u16 fsb = sysinfo->fsb_frequency;
@ -1691,17 +1690,14 @@ static void sdram_program_graphics_frequency(struct sys_info *sysinfo)
if ((fsb == 667 && mem == 533) ||
(fsb == 533 && mem == 533) ||
(fsb == 533 && mem == 400)) {
second_vco = 1;
second_vco = true;
}
if (fsb == 667 && mem == 533)
sysinfo->mvco4x = 1;
}
if (second_vco)
sysinfo->clkcfg_bit7 = 1;
else
sysinfo->clkcfg_bit7 = 0;
sysinfo->clkcfg_bit7 = second_vco;
/* Graphics Core Render Clock */
pci_update_config16(IGD_DEV, GCFC, ~((7 << 0) | (1 << 13)), freq);

View File

@ -30,7 +30,7 @@ struct sys_info {
bool interleaved;
u8 mvco4x; /* 0 (8x) or 1 (4x) */
u8 clkcfg_bit7;
bool clkcfg_bit7;
u8 boot_path;
#define BOOT_PATH_NORMAL 0
#define BOOT_PATH_RESET 1