Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39350 )
Change subject: mb/asus/p8z77-m_pro: Conistently spell *PRO* in board name uppercase
......................................................................
mb/asus/p8z77-m_pro: Conistently spell *PRO* in board name uppercase
Change-Id: I2e2d62389d1b965f4a391080a10e7f97fa787d14
Signed-off-by: Paul Menzel <pmenzel(a)molgen.mpg.de>
---
M Documentation/mainboard/asus/p8z77-m_pro.md
M src/mainboard/asus/p8z77-m_pro/early_init.c
2 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/39350/1
diff --git a/Documentation/mainboard/asus/p8z77-m_pro.md b/Documentation/mainboard/asus/p8z77-m_pro.md
index 2d9f612..1101089 100644
--- a/Documentation/mainboard/asus/p8z77-m_pro.md
+++ b/Documentation/mainboard/asus/p8z77-m_pro.md
@@ -1,6 +1,6 @@
-# ASUS P8Z77-M Pro
+# ASUS P8Z77-M PRO
-This page describes how to run coreboot on the [ASUS P8Z77-M Pro]
+This page describes how to run coreboot on the [ASUS P8Z77-M PRO]
## Flashing coreboot
@@ -163,6 +163,6 @@
- [Flash chip datasheet][W25Q64FVA1Q]
-[ASUS P8Z77-M Pro]: https://www.asus.com/Motherboards/P8Z77M_PRO/
+[ASUS P8Z77-M PRO]: https://www.asus.com/Motherboards/P8Z77M_PRO/
[W25Q64FVA1Q]: https://www.winbond.com/resource-files/w25q64fv%20revs%2007182017.pdf
[flashrom]: https://flashrom.org/Flashrom
diff --git a/src/mainboard/asus/p8z77-m_pro/early_init.c b/src/mainboard/asus/p8z77-m_pro/early_init.c
index 6c76a7a..d2c2355 100644
--- a/src/mainboard/asus/p8z77-m_pro/early_init.c
+++ b/src/mainboard/asus/p8z77-m_pro/early_init.c
@@ -154,7 +154,7 @@
*/
usb3_streams
},
- /* ASUS P8Z77-M Pro manual says 1.35v DIMMs are supported */
+ /* ASUS P8Z77-M PRO manual says 1.35v DIMMs are supported */
.ddr3lv_support = 1,
/* PCIe 3.0 support. As we use Ivy Bridge, let's enable it,
* but might cause some system instability !
--
To view, visit https://review.coreboot.org/c/coreboot/+/39350
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2e2d62389d1b965f4a391080a10e7f97fa787d14
Gerrit-Change-Number: 39350
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newchange
Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39349 )
Change subject: Docs: Fix link for ASUS P8Z77-M Pro
......................................................................
Docs: Fix link for ASUS P8Z77-M Pro
Change-Id: I2b8ff31acc7da2b1ded036604fa4a6b6d6d9cac0
Signed-off-by: Paul Menzel <pmenzel(a)molgen.mpg.de>
---
M Documentation/mainboard/asus/p8z77-m_pro.md
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/39349/1
diff --git a/Documentation/mainboard/asus/p8z77-m_pro.md b/Documentation/mainboard/asus/p8z77-m_pro.md
index 7c84149..2d9f612 100644
--- a/Documentation/mainboard/asus/p8z77-m_pro.md
+++ b/Documentation/mainboard/asus/p8z77-m_pro.md
@@ -163,6 +163,6 @@
- [Flash chip datasheet][W25Q64FVA1Q]
-[ASUS P8Z88-M Pro]: https://www.asus.com/Motherboards/P8Z77M_PRO/
+[ASUS P8Z77-M Pro]: https://www.asus.com/Motherboards/P8Z77M_PRO/
[W25Q64FVA1Q]: https://www.winbond.com/resource-files/w25q64fv%20revs%2007182017.pdf
[flashrom]: https://flashrom.org/Flashrom
--
To view, visit https://review.coreboot.org/c/coreboot/+/39349
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2b8ff31acc7da2b1ded036604fa4a6b6d6d9cac0
Gerrit-Change-Number: 39349
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newchange
Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39456 )
Change subject: soc/intel/icelake: Correct past participle in comment
......................................................................
soc/intel/icelake: Correct past participle in comment
Change-Id: I117c8d2f71824292c4ca87b6f9434d2106bb512d
Signed-off-by: Paul Menzel <pmenzel(a)molgen.mpg.de>
---
M src/soc/intel/icelake/romstage/fsp_params.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/39456/1
diff --git a/src/soc/intel/icelake/romstage/fsp_params.c b/src/soc/intel/icelake/romstage/fsp_params.c
index 8dd6bfd..99f606b 100644
--- a/src/soc/intel/icelake/romstage/fsp_params.c
+++ b/src/soc/intel/icelake/romstage/fsp_params.c
@@ -32,7 +32,7 @@
if (!dev || !dev->enabled) {
/*
* Skip IGD initialization in FSP if device
- * is disable in devicetree.cb.
+ * is disabled in devicetree.cb.
*/
m_cfg->InternalGfx = 0;
m_cfg->IgdDvmt50PreAlloc = 0;
--
To view, visit https://review.coreboot.org/c/coreboot/+/39456
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I117c8d2f71824292c4ca87b6f9434d2106bb512d
Gerrit-Change-Number: 39456
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newchange
EricR Lai has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39496 )
Change subject: lib/spd_bin: Cleanup spd_get_banks
......................................................................
lib/spd_bin: Cleanup spd_get_banks
Remove the switch case in spd_get_banks. The new DDR type still adapt
DDR4 attributes.
Signed-off-by: Eric Lai <ericr_lai(a)compal.corp-partner.google.com>
Change-Id: Icfaefd1856d2350c6e5a91d233ccdb10d5259391
---
M src/lib/spd_bin.c
1 file changed, 6 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/39496/1
diff --git a/src/lib/spd_bin.c b/src/lib/spd_bin.c
index 625eec8..1300ac8 100644
--- a/src/lib/spd_bin.c
+++ b/src/lib/spd_bin.c
@@ -78,22 +78,15 @@
static const int ddr3_banks[4] = { 8, 16, 32, 64 };
static const int ddr4_banks[10] = { 4, 8, -1, -1, 8, 16, -1, -1, 16, 32 };
int index = (spd[SPD_DENSITY_BANKS] >> 4) & 0xf;
- switch (dram_type) {
- /* DDR3 and LPDDR3_Intel have the same bank definition */
- case SPD_DRAM_DDR3:
- case SPD_DRAM_LPDDR3_INTEL:
- if (index >= ARRAY_SIZE(ddr3_banks))
- return -1;
- return ddr3_banks[index];
- /* LPDDR3, LPDDR4 and DDR4 have the same bank definition */
- case SPD_DRAM_LPDDR3_JEDEC:
- case SPD_DRAM_DDR4:
- case SPD_DRAM_LPDDR4:
+
+ if (use_ddr4_params(dram_type)) {
if (index >= ARRAY_SIZE(ddr4_banks))
return -1;
return ddr4_banks[index];
- default:
- return -1;
+ } else {
+ if (index >= ARRAY_SIZE(ddr3_banks))
+ return -1;
+ return ddr3_banks[index];
}
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/39496
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Icfaefd1856d2350c6e5a91d233ccdb10d5259391
Gerrit-Change-Number: 39496
Gerrit-PatchSet: 1
Gerrit-Owner: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-MessageType: newchange