Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40581 )
Change subject: mainboard/clevo/n141cu: Add new mainboard
......................................................................
Patch Set 27:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@26
PS27, Line 26: - All of the hardware above
> Sorry, I thought, Clevo does not sell directly, and this is a, for example, TUXEDO device.
It is a TUXEDO notebook indeed :) I will add that for reference later.
--
To view, visit https://review.coreboot.org/c/coreboot/+/40581
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9f83fab64e4cc9036698ca0fdd5edbb677d77eb9
Gerrit-Change-Number: 40581
Gerrit-PatchSet: 27
Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 06 May 2020 11:29:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Singer <felixsinger(a)posteo.net>
Comment-In-Reply-To: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: comment
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40581 )
Change subject: mainboard/clevo/n141cu: Add new mainboard
......................................................................
Patch Set 27:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@26
PS27, Line 26: - All of the hardware above
> What do you mean? I assume you're missing "Clevo N141CU" here.
Sorry, I thought, Clevo does not sell directly, and this is a, for example, TUXEDO device.
--
To view, visit https://review.coreboot.org/c/coreboot/+/40581
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9f83fab64e4cc9036698ca0fdd5edbb677d77eb9
Gerrit-Change-Number: 40581
Gerrit-PatchSet: 27
Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 06 May 2020 11:04:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Singer <felixsinger(a)posteo.net>
Comment-In-Reply-To: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: comment
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40581 )
Change subject: mainboard/clevo/n141cu: Add new mainboard
......................................................................
Patch Set 27:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@26
PS27, Line 26: - All of the hardware above
> What model/brand?
What do you mean? I assume you're missing "Clevo N141CU" here.
--
To view, visit https://review.coreboot.org/c/coreboot/+/40581
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9f83fab64e4cc9036698ca0fdd5edbb677d77eb9
Gerrit-Change-Number: 40581
Gerrit-PatchSet: 27
Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
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: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 06 May 2020 10:57:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: comment
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41078 )
Change subject: mb/google/volteer: Enable EARLY_EC_SYNC
......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/41078/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/41078/1//COMMIT_MSG@9
PS1, Line 9: This enables EC software sync in romstage.
… to update the EC early on to fix …?
https://review.coreboot.org/c/coreboot/+/41078/1//COMMIT_MSG@11
PS1, Line 11: BUG=b:148259137
Please summarize the bug report.
https://review.coreboot.org/c/coreboot/+/41078/1//COMMIT_MSG@12
PS1, Line 12: TEST=verified EC is updated in romstage using serial BIOS
What is “serial BIOS”?
> looking at coreboot’s messages on the serial console?
--
To view, visit https://review.coreboot.org/c/coreboot/+/41078
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibb97c1d57220f7fd74131a5aee450b1ab4b1c982
Gerrit-Change-Number: 41078
Gerrit-PatchSet: 1
Gerrit-Owner: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: Daisuke Nojiri <dnojiri(a)chromium.org>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Wed, 06 May 2020 09:45:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Jamie Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40415 )
Change subject: mb/google/puff: add a region to cache SPD data
......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40415/2/src/mainboard/google/hatch…
File src/mainboard/google/hatch/Kconfig:
https://review.coreboot.org/c/coreboot/+/40415/2/src/mainboard/google/hatch…
PS2, Line 83: OARD_GOOGLE_PUFF
> Could this be: […]
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/40415
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8d07fddf113a767d62394cb31e33b56f22f74351
Gerrit-Change-Number: 40415
Gerrit-PatchSet: 7
Gerrit-Owner: Jamie Chen <jamie.chen(a)intel.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Kane Chen <kane.chen(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-CC: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Wed, 06 May 2020 08:53:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-MessageType: comment
Jamie Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40415 )
Change subject: mb/google/puff: add a region to cache SPD data
......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/40415/2//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/40415/2//COMMIT_MSG@10
PS2, Line 10: the boot time
> What is cached?
SPD data of SO-DIMM.
https://review.coreboot.org/c/coreboot/+/40415/2//COMMIT_MSG@10
PS2, Line 10: can use
> can be used
Done
https://review.coreboot.org/c/coreboot/+/40415/2//COMMIT_MSG@11
PS2, Line 11: is be changed
> is changed
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/40415
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8d07fddf113a767d62394cb31e33b56f22f74351
Gerrit-Change-Number: 40415
Gerrit-PatchSet: 7
Gerrit-Owner: Jamie Chen <jamie.chen(a)intel.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Kane Chen <kane.chen(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-CC: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Wed, 06 May 2020 08:52:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: comment