Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29967 )
Change subject: qclib: Add qclib support with interface tables
......................................................................
Patch Set 30:
(2 comments)
https://review.coreboot.org/#/c/29967/30/src/mainboard/google/mistral/romst…
File src/mainboard/google/mistral/romstage.c:
https://review.coreboot.org/#/c/29967/30/src/mainboard/google/mistral/romst…
PS30, Line 32: qclib_load_and_run();
Normally this should be the last call in this function.
https://review.coreboot.org/#/c/29967/30/src/soc/qualcomm/common/qclib.c
File src/soc/qualcomm/common/qclib.c:
https://review.coreboot.org/#/c/29967/30/src/soc/qualcomm/common/qclib.c@147
PS30, Line 147: _ddr_training, ssize, QCLIB_BA_SAVE_TO_STORAGE);
Still wrong
--
To view, visit https://review.coreboot.org/c/coreboot/+/29967
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I534af71163d034ea04420dda6a94ce31b08c8a07
Gerrit-Change-Number: 29967
Gerrit-PatchSet: 30
Gerrit-Owner: Nitheesh Sekar <nsekar(a)codeaurora.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nitheesh Sekar <nsekar(a)codeaurora.org>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Sricharan Ramabadhran <srichara(a)qualcomm.corp-partner.google.com>
Gerrit-CC: mturney mturney <mturney(a)codeaurora.org>
Gerrit-Comment-Date: Mon, 17 Jun 2019 21:38:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Marty E. Plummer has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33486
Change subject: arm64: make comment match code
......................................................................
arm64: make comment match code
Change-Id: I37b8d5715cb6a32d4853e77098094cd5cffb9a4c
Signed-off-by: Marty E. Plummer <hanetzer(a)startmail.com>
---
M src/arch/arm64/armv8/bootblock.S
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/33486/1
diff --git a/src/arch/arm64/armv8/bootblock.S b/src/arch/arm64/armv8/bootblock.S
index e5758bc..3950f55 100644
--- a/src/arch/arm64/armv8/bootblock.S
+++ b/src/arch/arm64/armv8/bootblock.S
@@ -19,7 +19,7 @@
/* NOTE: When making changes to general ARM64 initialization, keep in mind that
* there are other CPU entry points, using BOOTBLOCK_CUSTOM or entering the CPU
* in a later stage (like Tegra). Changes should generally be put into
- * arm64_cpu_init so they can be shared between those instances. */
+ * arm64_init_cpu so they can be shared between those instances. */
ENTRY(_start)
/* Initialize PSTATE, SCTLR and caches to clean state, set up stack. */
--
To view, visit https://review.coreboot.org/c/coreboot/+/33486
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I37b8d5715cb6a32d4853e77098094cd5cffb9a4c
Gerrit-Change-Number: 33486
Gerrit-PatchSet: 1
Gerrit-Owner: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-MessageType: newchange
Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29662 )
Change subject: {drivers,soc/intel/braswell}: Implement C_ENVIRONMENT_BOOTBLOCK support
......................................................................
Patch Set 47:
this change breaks google/cyan variants, bootloops and no console output to triage (even with BOOTBLOCK_CONSOLE added)
--
To view, visit https://review.coreboot.org/c/coreboot/+/29662
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iab48ad72f1514c93f20d70db5ef4fd8fa2383e8c
Gerrit-Change-Number: 29662
Gerrit-PatchSet: 47
Gerrit-Owner: Frans Hendriks <fhendriks(a)eltan.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Frans Hendriks <fhendriks(a)eltan.com>
Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-Reviewer: Huang Jin <huang.jin(a)intel.com>
Gerrit-Reviewer: Lee Leahy <leroy.p.leahy(a)intel.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: York Yang <yyang024(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Mon, 17 Jun 2019 17:34:18 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Reto Buerki has posted comments on this change. ( https://review.coreboot.org/c/libgfxinit/+/27154 )
Change subject: gma pipe setup: Fix secondary pipe cursors <= Sandy Bridge
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/27154/3/common/hw-gfx-gma-pipe_setup.ads
File common/hw-gfx-gma-pipe_setup.ads:
https://review.coreboot.org/#/c/27154/3/common/hw-gfx-gma-pipe_setup.ads@244
PS3, Line 244: FBC_CTL : Registers.Registers_Invalid_Index;
> Everything is aligned to a multiple of 3 here. I don't have a written […]
Ah ic, I did not know that it is aligned to a multiple of 3. I'm not aware of an internal style guide which applies here.
--
To view, visit https://review.coreboot.org/c/libgfxinit/+/27154
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: libgfxinit
Gerrit-Branch: master
Gerrit-Change-Id: I4d79f59a8cb693d73d6da666525f091021efb4fd
Gerrit-Change-Number: 27154
Gerrit-PatchSet: 3
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Adrian-Ken Rueegsegger <ken(a)codelabs.ch>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Reto Buerki <reet(a)codelabs.ch>
Gerrit-Comment-Date: Mon, 17 Jun 2019 13:29:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Nico Huber <nico.h(a)gmx.de>
Comment-In-Reply-To: Reto Buerki <reet(a)codelabs.ch>
Gerrit-MessageType: comment
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/libgfxinit/+/27154 )
Change subject: gma pipe setup: Fix secondary pipe cursors <= Sandy Bridge
......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/#/c/27154/3/common/hw-gfx-gma-pipe_setup.ads
File common/hw-gfx-gma-pipe_setup.ads:
https://review.coreboot.org/#/c/27154/3/common/hw-gfx-gma-pipe_setup.ads@244
PS3, Line 244: FBC_CTL : Registers.Registers_Invalid_Index;
> Minor: Remove one whitespace before :?
Everything is aligned to a multiple of 3 here. I don't have a written
style guide, do you (or do we, internally)?
--
To view, visit https://review.coreboot.org/c/libgfxinit/+/27154
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: libgfxinit
Gerrit-Branch: master
Gerrit-Change-Id: I4d79f59a8cb693d73d6da666525f091021efb4fd
Gerrit-Change-Number: 27154
Gerrit-PatchSet: 3
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Adrian-Ken Rueegsegger <ken(a)codelabs.ch>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Reto Buerki <reet(a)codelabs.ch>
Gerrit-Comment-Date: Mon, 17 Jun 2019 13:18:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Reto Buerki <reet(a)codelabs.ch>
Gerrit-MessageType: comment