Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35308 )
Change subject: mb/google/poppy/variants/rammus: Add workaround for touchscreen FW recovery
......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35308/2//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/35308/2//COMMIT_MSG@7
PS2, Line 7: mb/google/poppy/variants/rammus: Add workaround for touchscreen FW recovery
Please be specific:
> Increase touchscreen power on delay to 1.5 s
https://review.coreboot.org/c/coreboot/+/35308/2//COMMIT_MSG@9
PS2, Line 9: Touchscreen controller need more time for initialization
: when fw main code is broken and executing firmware bootloader error
: recovery mechanism.
I do not understand the description. Please rephrase the problem.
https://review.coreboot.org/c/coreboot/+/35308/2//COMMIT_MSG@8
PS2, Line 8:
: Touchscreen controller need more time for initialization
: when fw main code is broken and executing firmware bootloader error
: recovery mechanism.
: Increase touchscreen controller power on delay time to postpone
: host access touchscreen controller.
Please use a text width of 75 characters, and add a blank line between paragraphs.
--
To view, visit https://review.coreboot.org/c/coreboot/+/35308
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iccbc3da55c41b93c9c7faada532a1d849eab2a86
Gerrit-Change-Number: 35308
Gerrit-PatchSet: 2
Gerrit-Owner: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Zhuohao Lee <zhuohao(a)chromium.org>
Gerrit-CC: Kane Chen <kane_chen(a)pegatron.corp-partner.google.com>
Gerrit-CC: Ken Lu <ken_lu(a)pegatron.corp-partner.google.com>
Gerrit-CC: Matthew Blecker <matthewb(a)chromium.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 24 Sep 2019 11:01:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Alexander Couzens has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35391 )
Change subject: Makefile: set TZ LANG LC_ALL
......................................................................
Makefile: set TZ LANG LC_ALL
To be reproducible, TZ LANG LC_ALL should be set early
in the build process to be always used.
Change-Id: Iad802968347c8d41f974af930e0d0ad5b66719cb
---
M Makefile
1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/35391/1
diff --git a/Makefile b/Makefile
index f0a22a5..deb80a4 100644
--- a/Makefile
+++ b/Makefile
@@ -39,6 +39,11 @@
objk := $(objutil)/kconfig
absobj := $(abspath $(obj))
+LC_ALL:=C
+LANG:=C
+TZ:=UTC0
+export LC_ALL LANG TZ
+
COREBOOT_EXPORTS := COREBOOT_EXPORTS
COREBOOT_EXPORTS += top src srck obj objutil objk
--
To view, visit https://review.coreboot.org/c/coreboot/+/35391
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iad802968347c8d41f974af930e0d0ad5b66719cb
Gerrit-Change-Number: 35391
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-MessageType: newchange
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/35322 )
Change subject: mainboard/ocp/monolake: Hide internal NIC
......................................................................
mainboard/ocp/monolake: Hide internal NIC
Disable root port IOU0 to which built-in NIC is attached.
TEST=on OCP monolake, hide built-in NIC and make sure OS does not report
built-in NIC
Change-Id: I2384e7dd073355f0ced2902ac2d8418996b1c5aa
Signed-off-by: Andrey Petrov <anpetrov(a)fb.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35322
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Patrick Rudolph <siro(a)das-labor.org>
Reviewed-by: David Hendricks <david.hendricks(a)gmail.com>
---
M src/mainboard/ocp/monolake/devicetree.cb
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
build bot (Jenkins): Verified
David Hendricks: Looks good to me, approved
Patrick Rudolph: Looks good to me, approved
diff --git a/src/mainboard/ocp/monolake/devicetree.cb b/src/mainboard/ocp/monolake/devicetree.cb
index 6a8bef1..26c95d5 100644
--- a/src/mainboard/ocp/monolake/devicetree.cb
+++ b/src/mainboard/ocp/monolake/devicetree.cb
@@ -4,6 +4,8 @@
end
device domain 0 on
device pci 00.0 on end # SoC router
+ device pci 02.2 off end # IOU0 port C, 10GbE
+ device pci 02.3 off end # IOU0 port D, 10GbE
device pci 14.0 on end # xHCI Controller
device pci 19.0 on end # Gigabit LAN Controller
device pci 1d.0 on end # EHCI Controller
--
To view, visit https://review.coreboot.org/c/coreboot/+/35322
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2384e7dd073355f0ced2902ac2d8418996b1c5aa
Gerrit-Change-Number: 35322
Gerrit-PatchSet: 11
Gerrit-Owner: Andrey Petrov <anpetrov(a)fb.com>
Gerrit-Reviewer: Andrey Petrov <anpetrov(a)fb.com>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged