Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28648 )
Change subject: nb/intel/i945: Set CxDRT1 tRPALL bit if populated with 8-bank
......................................................................
Patch Set 7: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/28648/7/src/northbridge/intel/i945/raminit.c
File src/northbridge/intel/i945/raminit.c:
https://review.coreboot.org/#/c/28648/7/src/northbridge/intel/i945/raminit.…
PS7, …
[View More]Line 1495: if (sysinfo->banks[i] != 8)
: continue;
: temp_drt |= (1 << 16);
Since we only want to set this bit once, why don't we do the following:
if (sysinfo->banks[i] == 8) {
temp_drt |= (1 << 16);
break;
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/28648
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id6c7dccd295e187acfe00a08294010af53b4d0ee
Gerrit-Change-Number: 28648
Gerrit-PatchSet: 7
Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 Jan 2019 12:41:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
[View Less]
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30331 )
Change subject: google/kukui: Move some initialization from bootblock to verstage.
......................................................................
Patch Set 12: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/30331/12//COMMIT_MSG
Commit Message:
https://review.coreboot.org/#/c/30331/12//COMMIT_MSG@7
PS12, Line 7: google/kukui: Move some initialization from bootblock to …
[View More]verstage.
Please remove the dot/period at the end of the commit message summary.
--
To view, visit https://review.coreboot.org/c/coreboot/+/30331
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9968d88c54283ef334d1ab975086d4adb3363bd6
Gerrit-Change-Number: 30331
Gerrit-PatchSet: 12
Gerrit-Owner: You-Cheng Syu <youcheng(a)google.com>
Gerrit-Reviewer: Chun-ta Lin <itspeter(a)google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Tristan Hsieh <tristan.shieh(a)mediatek.com>
Gerrit-Reviewer: You-Cheng Syu <youcheng(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-Comment-Date: Mon, 28 Jan 2019 12:12:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
[View Less]
Lijian Zhao has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31112
Change subject: soc/intel/common/rtc: Enable RTC if common code RTC
......................................................................
soc/intel/common/rtc: Enable RTC if common code RTC
Intel RTC common code driver need to turn on RTC itself.
BUG=b:123372643
TEST=build and boot up on sarien platform, check .config to see
CONFIG_RTC is set.
Signed-off-by: Lijian Zhao <lijian.zhao(a)…
[View More]intel.com>
Change-Id: I0c8c5cf9e6f7f338b1f2f784c04254649d257536
---
M src/soc/intel/common/block/rtc/Kconfig
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/31112/1
diff --git a/src/soc/intel/common/block/rtc/Kconfig b/src/soc/intel/common/block/rtc/Kconfig
index d194f15..eda2063 100644
--- a/src/soc/intel/common/block/rtc/Kconfig
+++ b/src/soc/intel/common/block/rtc/Kconfig
@@ -1,4 +1,5 @@
config SOC_INTEL_COMMON_BLOCK_RTC
bool
+ select RTC
help
Intel Processor common RTC support
--
To view, visit https://review.coreboot.org/c/coreboot/+/31112
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0c8c5cf9e6f7f338b1f2f784c04254649d257536
Gerrit-Change-Number: 31112
Gerrit-PatchSet: 1
Gerrit-Owner: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-MessageType: newchange
[View Less]
Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30331 )
Change subject: google/kukui: Move some initialization from bootblock to verstage.
......................................................................
Patch Set 12: Code-Review+1
--
To view, visit https://review.coreboot.org/c/coreboot/+/30331
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
…
[View More]Gerrit-Change-Id: I9968d88c54283ef334d1ab975086d4adb3363bd6
Gerrit-Change-Number: 30331
Gerrit-PatchSet: 12
Gerrit-Owner: You-Cheng Syu <youcheng(a)google.com>
Gerrit-Reviewer: Chun-ta Lin <itspeter(a)google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Tristan Hsieh <tristan.shieh(a)mediatek.com>
Gerrit-Reviewer: You-Cheng Syu <youcheng(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-Comment-Date: Mon, 28 Jan 2019 10:00:51 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
[View Less]
Ronak Kanabar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31095 )
Change subject: src/soc/intel/cnl/chip.h: Fix preprocessor condition
......................................................................
Patch Set 5:
> Patch Set 4:
>
> we will take this clean up activity to remove hard coding for sagv config in devicetree.cb and make use of macro as applicable. That would helped us to catch this issue on WHLRVP in absence of hatch board in lab.
…
[View More]
I will push patch for hard coding clean up.
--
To view, visit https://review.coreboot.org/c/coreboot/+/31095
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8a6959bb615fb5668cbfe54339747d135bd5a005
Gerrit-Change-Number: 31095
Gerrit-PatchSet: 5
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 Jan 2019 06:39:48 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
[View Less]
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31095 )
Change subject: src/soc/intel/cnl/chip.h: Fix preprocessor condition
......................................................................
Patch Set 4:
we will take this clean up activity to remove hard coding for sagv config in devicetree.cb and make use of macro as applicable. That would helped us to catch this issue on WHLRVP in absence of hatch board in lab.
--
To view, visit https://…
[View More]review.coreboot.org/c/coreboot/+/31095
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8a6959bb615fb5668cbfe54339747d135bd5a005
Gerrit-Change-Number: 31095
Gerrit-PatchSet: 4
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 Jan 2019 06:36:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
[View Less]