Lucas Chen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44107 )
Change subject: mb/google/zork/ezkinil: Fix ELAN touchscreen ACPI node
......................................................................
mb/google/zork/ezkinil: Fix ELAN touchscreen ACPI node
a. USI_REPORT_EN is no longer set to high in coreboot. Instead
GPIO_144 is exposed as stop_gpio in ACPI to allow OS to control this
pad as required.
b. Add Delay after stop_gpio Low - 300ms
BUG=b:162263398
Signed-off-by: Lucas Chen <lucas.chen(a)quanta.corp-partner.google.com>
Change-Id: I3d4dcb6e5cae5d9515abfd415315ec4114ca80b4
---
M src/mainboard/google/zork/variants/ezkinil/overridetree.cb
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/44107/1
diff --git a/src/mainboard/google/zork/variants/ezkinil/overridetree.cb b/src/mainboard/google/zork/variants/ezkinil/overridetree.cb
index a4cdac6..1f97d1c 100644
--- a/src/mainboard/google/zork/variants/ezkinil/overridetree.cb
+++ b/src/mainboard/google/zork/variants/ezkinil/overridetree.cb
@@ -79,6 +79,8 @@
register "generic.irq_gpio" = "ACPI_GPIO_IRQ_EDGE_LOW(GPIO_12)"
register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_140)"
register "generic.reset_delay_ms" = "20"
+ register "generic.stop_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_144)"
+ register "generic.stop_delay_ms" = "300"
register "generic.has_power_resource" = "1"
register "hid_desc_reg_offset" = "0x01"
device i2c 10 on end
--
To view, visit https://review.coreboot.org/c/coreboot/+/44107
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3d4dcb6e5cae5d9515abfd415315ec4114ca80b4
Gerrit-Change-Number: 44107
Gerrit-PatchSet: 1
Gerrit-Owner: Lucas Chen <lucas.chen(a)quanta.corp-partner.google.com>
Gerrit-MessageType: newchange
HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44173 )
Change subject: src: Use space after 'if', 'for'
......................................................................
src: Use space after 'if', 'for'
Change-Id: I5d3a5ede47aefc7cc2ee330f8a0bcded16138764
Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr>
---
M src/device/oprom/realmode/x86.c
M src/mainboard/google/peach_pit/mainboard.c
M src/mainboard/google/peach_pit/romstage.c
M src/mainboard/roda/rk886ex/m3885.c
M src/northbridge/intel/x4x/dq_dqs.c
M src/northbridge/intel/x4x/raminit_ddr23.c
M src/soc/intel/skylake/sd.c
7 files changed, 9 insertions(+), 9 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/44173/1
diff --git a/src/device/oprom/realmode/x86.c b/src/device/oprom/realmode/x86.c
index 5215934..d230d5c 100644
--- a/src/device/oprom/realmode/x86.c
+++ b/src/device/oprom/realmode/x86.c
@@ -130,7 +130,7 @@
{
/* If the mainboard_interrupt_handler isn't called first.
*/
- if(!intXX_handler[i])
+ if (!intXX_handler[i])
{
/* Now set the default functions that are actually
* needed to initialize the option roms. This is
diff --git a/src/mainboard/google/peach_pit/mainboard.c b/src/mainboard/google/peach_pit/mainboard.c
index 8b54b99..33541b2 100644
--- a/src/mainboard/google/peach_pit/mainboard.c
+++ b/src/mainboard/google/peach_pit/mainboard.c
@@ -254,7 +254,7 @@
* we're out of here.
* If it's not ready after a second, then we're in big trouble.
*/
- for(i = 0; i < 1000; i++){
+ for (i = 0; i < 1000; i++){
if (gpio_get_value(dp_hpd))
break;
mdelay(1);
diff --git a/src/mainboard/google/peach_pit/romstage.c b/src/mainboard/google/peach_pit/romstage.c
index d99db80..53fdfba 100644
--- a/src/mainboard/google/peach_pit/romstage.c
+++ b/src/mainboard/google/peach_pit/romstage.c
@@ -129,13 +129,13 @@
unsigned long *l = (void *)0x40000000;
int bad = 0;
unsigned long i;
- for(i = 0; i < 256*1048576; i++){
+ for (i = 0; i < 256*1048576; i++){
if (! (i%1048576))
printk(BIOS_SPEW, "%lu ...", i);
l[i] = 0xffffffff - i;
}
- for(i = 0; i < 256*1048576; i++){
+ for (i = 0; i < 256*1048576; i++){
if (! (i%1048576))
printk(BIOS_SPEW, "%lu ...", i);
if (l[i] != (0xffffffff - i)){
@@ -177,7 +177,7 @@
}
- for(i = 0; i < amt; i += 4){
+ for (i = 0; i < amt; i += 4){
if (rdev_readat(boot_dev, &in, i, 4) < 4) {
printk(BIOS_SPEW, "simple_spi_test fails at %d\n", i);
return;
diff --git a/src/mainboard/roda/rk886ex/m3885.c b/src/mainboard/roda/rk886ex/m3885.c
index 6756e0e..59dd575 100644
--- a/src/mainboard/roda/rk886ex/m3885.c
+++ b/src/mainboard/roda/rk886ex/m3885.c
@@ -251,7 +251,7 @@
maxvars = m3885_get_variable(0x00);
printk(BIOS_DEBUG, "M388x has %d variables in original bank.\n", maxvars);
for (i = 0; i < ARRAY_SIZE(variables); i+=3) {
- if(variables[i + 0] > maxvars)
+ if (variables[i + 0] > maxvars)
continue;
reg8 = m3885_get_variable(variables[i + 0]);
reg8 &= ~(variables[i + 1]);
diff --git a/src/northbridge/intel/x4x/dq_dqs.c b/src/northbridge/intel/x4x/dq_dqs.c
index 52166ed..489340a 100644
--- a/src/northbridge/intel/x4x/dq_dqs.c
+++ b/src/northbridge/intel/x4x/dq_dqs.c
@@ -498,7 +498,7 @@
FOR_EACH_BYTELANE(lane) {
saved_dqs_center[channel][lane] /= RT_LOOPS;
while (saved_dqs_center[channel][lane]--) {
- if(rt_increment_dqs(&s->rt_dqs[channel][lane])
+ if (rt_increment_dqs(&s->rt_dqs[channel][lane])
== CB_ERR)
/* Should never happen */
printk(BIOS_ERR,
diff --git a/src/northbridge/intel/x4x/raminit_ddr23.c b/src/northbridge/intel/x4x/raminit_ddr23.c
index 332af32..57d5ef4 100644
--- a/src/northbridge/intel/x4x/raminit_ddr23.c
+++ b/src/northbridge/intel/x4x/raminit_ddr23.c
@@ -2166,7 +2166,7 @@
if (!fast_boot) {
if (s->selected_timings.mem_clk > MEM_CLOCK_667MHz) {
- if(do_write_training(s))
+ if (do_write_training(s))
die("DQ write training failed!");
}
if (do_read_training(s))
diff --git a/src/soc/intel/skylake/sd.c b/src/soc/intel/skylake/sd.c
index 1f40d98..ba56182 100644
--- a/src/soc/intel/skylake/sd.c
+++ b/src/soc/intel/skylake/sd.c
@@ -8,7 +8,7 @@
config_t *config = config_of(dev);
/* Nothing to write if GPIO is not set in devicetree */
- if(!config->sdcard_cd_gpio_default && !config->sdcard_cd_gpio.pins[0])
+ if (!config->sdcard_cd_gpio_default && !config->sdcard_cd_gpio.pins[0])
return -1;
if (config->sdcard_cd_gpio_default) {
--
To view, visit https://review.coreboot.org/c/coreboot/+/44173
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5d3a5ede47aefc7cc2ee330f8a0bcded16138764
Gerrit-Change-Number: 44173
Gerrit-PatchSet: 1
Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-MessageType: newchange
Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36356 )
Change subject: soc/intel/cannonlake: set LT_LOCK_MEMORY at end of POST
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36356/2/src/soc/intel/cannonlake/f…
File src/soc/intel/cannonlake/finalize.c:
https://review.coreboot.org/c/coreboot/+/36356/2/src/soc/intel/cannonlake/f…
PS2, Line 116: mp_run_on_all_cpus
> For #1) Your right that LT_LOCK_MEMORY is package scoped. […]
@Nate thank you very much!
--
To view, visit https://review.coreboot.org/c/coreboot/+/36356
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iaadd4996653c4f27d268b1c4773c1e2e86114912
Gerrit-Change-Number: 36356
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Niewöhner
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Michael Niewöhner
Gerrit-Reviewer: Nathaniel L Desimone <nathaniel.l.desimone(a)intel.com>
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: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Gerrit-Reviewer: Varun Joshi <varun.joshi(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Wed, 05 Aug 2020 09:32:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Comment-In-Reply-To: Michael Niewöhner
Comment-In-Reply-To: Nathaniel L Desimone <nathaniel.l.desimone(a)intel.com>
Gerrit-MessageType: comment
Andrew McRae has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/37881 )
Change subject: mainboard/google/puff Add device_index for NIC
......................................................................
Abandoned
--
To view, visit https://review.coreboot.org/c/coreboot/+/37881
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0c508139757c9e92986a9c83b1dd7dc05bdbefd6
Gerrit-Change-Number: 37881
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew McRae <amcrae(a)chromium.org>
Gerrit-Reviewer: Andrew McRae <amcrae(a)chromium.org>
Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-Reviewer: Kangheui Won <khwon(a)chromium.org>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Kangheui Won <khwon(a)google.com>
Gerrit-MessageType: abandon