Attention is currently required from: Henry Sun, Paul Menzel, Karthik Ramasubramanian.
Tao Xia has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55035 )
Change subject: mb/google/dedede/var/sasukette: Modify the touch pad slave address
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/55035/comment/c9ec1e6e_5b5620c1
PS1, Line 17: Change-Id: I09bc12eeca7f34f6e9424e5cbb7f812f3bbfe0de
> Please remove this line, as the Change-Id below is used. (Please also remove the blank line. […]
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/55035
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If0bd80baa27dfeb7bcb43f0ca4b02e1228e372a6
Gerrit-Change-Number: 55035
Gerrit-PatchSet: 2
Gerrit-Owner: Tao Xia <xiatao5(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Henry Sun <henrysun(a)google.com>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jessie Xu <xuyanan(a)huaqin.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-CC: zanxi chen <chenzanxi(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Henry Sun <henrysun(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Comment-Date: Sun, 30 May 2021 06:28:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Jason Glenesk, Raul Rangel, Anjaneya "Reddy" Chagam, Marshall Dawson, Jonathan Zhang, Johnny Lin, Arthur Heymans, Morgan Jang, Patrick Rudolph, Felix Held.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55063 )
Change subject: use initial_lapicid() instead of opencoding it
......................................................................
Patch Set 1:
(1 comment)
File src/soc/intel/xeon_sp/smmrelocate.c:
https://review.coreboot.org/c/coreboot/+/55063/comment/020733c4_d86065f8
PS1, Line 90: int apic_id;
This can be removed now.
--
To view, visit https://review.coreboot.org/c/coreboot/+/55063
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I289015b81b2a9d915c4cab9b0544fc19b85df7a3
Gerrit-Change-Number: 55063
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Anjaneya "Reddy" Chagam <anjaneya.chagam(a)intel.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Reviewer: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Morgan Jang <Morgan_Jang(a)wiwynn.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Attention: Anjaneya "Reddy" Chagam <anjaneya.chagam(a)intel.com>
Gerrit-Attention: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Attention: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-Attention: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Morgan Jang <Morgan_Jang(a)wiwynn.com>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Sun, 30 May 2021 06:25:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Arthur Heymans, Kyösti Mälkki.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55074 )
Change subject: cbfstool/linux_trampoline.S: Fixup the e820 table
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/55074/comment/20669aea_c9204329
PS2, Line 7: Fixup
Fix up
https://review.coreboot.org/c/coreboot/+/55074/comment/697799dd_a15027b7
PS2, Line 13:
Maybe note, that Linux actually should treat unknown values as reserved.
--
To view, visit https://review.coreboot.org/c/coreboot/+/55074
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie0e41c66e002919e41590327afe0f543e0037369
Gerrit-Change-Number: 55074
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Comment-Date: Sun, 30 May 2021 06:23:43 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Jacob Garber.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55075 )
Change subject: include: always unsigned long for size_t
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Example build failure:
```
CC verstage/commonlib/bsd/cbfs_mcache.o
In file included from src/commonlib/bsd/include/commonlib/bsd/cbfs_private.h:42,
from src/commonlib/bsd/cbfs_mcache.c:4:
src/commonlib/bsd/cbfs_mcache.c: In function 'cbfs_mcache_build':
src/include/cbfs_glue.h:23:36: error: format '%zx' expects argument of type 'size_t', but argument 6 has type 'long unsigned int' [-Werror=format=]
#define LOG(...) printk(BIOS_INFO, "CBFS: " __VA_ARGS__)
^~~~~~~~
src/commonlib/bsd/cbfs_mcache.c:86:2: note: in expansion of macro 'LOG'
LOG("mcache @%p built for %d files, used %#zx of %#zx bytes\n", mcache,
^~~
src/commonlib/bsd/cbfs_mcache.c:86:54: note: format string is defined here
LOG("mcache @%p built for %d files, used %#zx of %#zx bytes\n", mcache,
~~~^
%#lx
```
The length modifier `z` also needs to be redefined?
Do you think it’s a good idea to divert from the use in other projects?
--
To view, visit https://review.coreboot.org/c/coreboot/+/55075
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8a8fa97c6961991dd8e291d2dc7d8762f83cad6d
Gerrit-Change-Number: 55075
Gerrit-PatchSet: 1
Gerrit-Owner: Jacob Garber <jgarber1(a)ualberta.ca>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Julius Werner <jwerner(a)chromium.org>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jacob Garber <jgarber1(a)ualberta.ca>
Gerrit-Comment-Date: Sun, 30 May 2021 06:21:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/55075 )
Change subject: include: always unsigned long for size_t
......................................................................
include: always unsigned long for size_t
Using the definition that GCC provides, size_t is an 'unsigned int' on
32-bit platforms and 'unsigned long' on 64-bit platforms. This is a
constant source of compatibility problems, since code is often written
for one integer type and then fails to compile for the other (e.g.
printf format specifiers). 'unsigned long' always matches the platform
size (see uintptr_t in <stdint.h>), so use it for size_t everywhere.
This also removes a weird macro hack for defining ssize_t.
Change-Id: I8a8fa97c6961991dd8e291d2dc7d8762f83cad6d
Signed-off-by: Jacob Garber <jgarber1(a)ualberta.ca>
---
M src/include/stddef.h
1 file changed, 3 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/75/55075/1
diff --git a/src/include/stddef.h b/src/include/stddef.h
index b668b1a..3e973e6 100644
--- a/src/include/stddef.h
+++ b/src/include/stddef.h
@@ -4,17 +4,9 @@
#include <commonlib/helpers.h>
typedef long ptrdiff_t;
-#ifndef __SIZE_TYPE__
-#define __SIZE_TYPE__ unsigned long
-#endif
-typedef __SIZE_TYPE__ size_t;
-/* There is a GCC macro for a size_t type, but not
- * for a ssize_t type. Below construct tricks GCC
- * into making __SIZE_TYPE__ signed.
- */
-#define unsigned signed
-typedef __SIZE_TYPE__ ssize_t;
-#undef unsigned
+
+typedef unsigned long size_t;
+typedef signed long ssize_t;
typedef int wchar_t;
typedef unsigned int wint_t;
--
To view, visit https://review.coreboot.org/c/coreboot/+/55075
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8a8fa97c6961991dd8e291d2dc7d8762f83cad6d
Gerrit-Change-Number: 55075
Gerrit-PatchSet: 1
Gerrit-Owner: Jacob Garber <jgarber1(a)ualberta.ca>
Gerrit-MessageType: newchange
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55072 )
Change subject: util/cbfstool/Makefile: Check out vboot before trying to use it
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://review.coreboot.org/c/coreboot/+/55072
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie2044d73b97663f7816964c4d908a46570aafdbc
Gerrit-Change-Number: 55072
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Idwer Vollering <vidwer(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sat, 29 May 2021 19:39:40 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Kyösti Mälkki.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55074 )
Change subject: cbfstool/linux_trampoline.S: Fixup the e820 table
......................................................................
Patch Set 2:
(12 comments)
File util/cbfstool/linux_trampoline.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/85f26bc2_e8630451
PS2, Line 6: 0xcb, 0x0f, 0x84, 0xaa, 0x00, 0x00, 0x00, 0x8b, 0x59, 0x04, 0x01, 0xcb, 0x8b, 0x49, 0x14, 0x83,
line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/d5d53de5_233009fa
PS2, Line 6: 0xcb, 0x0f, 0x84, 0xaa, 0x00, 0x00, 0x00, 0x8b, 0x59, 0x04, 0x01, 0xcb, 0x8b, 0x49, 0x14, 0x83,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/25b27ad4_7aee1f79
PS2, Line 7: 0x3b, 0x11, 0x75, 0x05, 0x8b, 0x4b, 0x08, 0xeb, 0xcf, 0x83, 0x3b, 0x01, 0x75, 0x53, 0x8b, 0x43,
line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/1ae15f43_b1e8aafd
PS2, Line 7: 0x3b, 0x11, 0x75, 0x05, 0x8b, 0x4b, 0x08, 0xeb, 0xcf, 0x83, 0x3b, 0x01, 0x75, 0x53, 0x8b, 0x43,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/0d385f87_5711edb4
PS2, Line 10: 0x00, 0x89, 0xf0, 0x91, 0x8d, 0x73, 0x08, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0xf3, 0xa5, 0x91, 0xa1,
line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/43d50bb4_4a0a6107
PS2, Line 10: 0x00, 0x89, 0xf0, 0x91, 0x8d, 0x73, 0x08, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0xf3, 0xa5, 0x91, 0xa1,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/369cee57_77568b65
PS2, Line 11: 0xe8, 0x01, 0x09, 0x00, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0x83, 0xf8, 0x00, 0x74, 0x18, 0x83, 0x7f,
line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/25554ef2_7bd4e3cd
PS2, Line 11: 0xe8, 0x01, 0x09, 0x00, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0x83, 0xf8, 0x00, 0x74, 0x18, 0x83, 0x7f,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/dedf4928_b0519a2c
PS2, Line 12: 0x10, 0x0c, 0x7e, 0x07, 0xc7, 0x47, 0x10, 0x02, 0x00, 0x00, 0x00, 0x48, 0x83, 0xc7, 0x14, 0xeb,
line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/444bdba4_1e29813f
PS2, Line 12: 0x10, 0x0c, 0x7e, 0x07, 0xc7, 0x47, 0x10, 0x02, 0x00, 0x00, 0x00, 0x48, 0x83, 0xc7, 0x14, 0xeb,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/69190673_46927d59
PS2, Line 13: 0xe8, 0x83, 0x3b, 0x12, 0x75, 0x00, 0x03, 0x5b, 0x04, 0x49, 0x75, 0x93, 0xb8, 0x00, 0x00, 0x04,
line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120288):
https://review.coreboot.org/c/coreboot/+/55074/comment/36df63ee_b43a938d
PS2, Line 13: 0xe8, 0x83, 0x3b, 0x12, 0x75, 0x00, 0x03, 0x5b, 0x04, 0x49, 0x75, 0x93, 0xb8, 0x00, 0x00, 0x04,
please, no spaces at the start of a line
--
To view, visit https://review.coreboot.org/c/coreboot/+/55074
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie0e41c66e002919e41590327afe0f543e0037369
Gerrit-Change-Number: 55074
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Comment-Date: Sat, 29 May 2021 18:54:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Kyösti Mälkki.
Hello build bot (Jenkins), Patrick Georgi, Kyösti Mälkki,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/55074
to look at the new patch set (#2).
Change subject: cbfstool/linux_trampoline.S: Fixup the e820 table
......................................................................
cbfstool/linux_trampoline.S: Fixup the e820 table
The e820 type don't fully match the LB_TAG_MEMORY types, so change all
unknown types to e820 to '2', reserved memory.
TESTED with Linuxboot: e820 now shows the CBMEM region as reserved.
Change-Id: Ie0e41c66e002919e41590327afe0f543e0037369
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M util/cbfstool/linux_trampoline.S
M util/cbfstool/linux_trampoline.c
2 files changed, 21 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/74/55074/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/55074
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie0e41c66e002919e41590327afe0f543e0037369
Gerrit-Change-Number: 55074
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-MessageType: newpatchset
Idwer Vollering has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55072 )
Change subject: util/cbfstool/Makefile: Check out vboot before trying to use it
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Patchset:
PS1:
I like it.
Build tested on and outside Ubuntu.
--
To view, visit https://review.coreboot.org/c/coreboot/+/55072
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie2044d73b97663f7816964c4d908a46570aafdbc
Gerrit-Change-Number: 55072
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Idwer Vollering <vidwer(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sat, 29 May 2021 18:42:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Kyösti Mälkki.
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55074 )
Change subject: cbfstool/linux_trampoline.S: Fixup the e820 table
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/55074
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie0e41c66e002919e41590327afe0f543e0037369
Gerrit-Change-Number: 55074
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Comment-Date: Sat, 29 May 2021 18:37:19 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment