Attention is currently required from: Martin Roth, Paul Menzel, Rex-BC Chen, Julius Werner, Marty E. Plummer.
Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58054 )
Change subject: payloads: Add UefiPayload support to ARM64 platforms
......................................................................
Patch Set 6:
(1 comment)
Patchset:
PS6:
> Do you mean you build fail for these two patches? […]
Marty, is this a generic failure in the build system that just for x86 because for most folks HOSTCC=CC=x86 gcc?
If so, can you help fix it?
--
To view, visit https://review.coreboot.org/c/coreboot/+/58054
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I10777a341d46240b91ceeeb1be26c33a0c5db839
Gerrit-Change-Number: 58054
Gerrit-PatchSet: 6
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:35:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Comment-In-Reply-To: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Martin Roth, Paul Menzel, Rex-BC Chen, Julius Werner, Marty E. Plummer.
Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58054 )
Change subject: payloads: Add UefiPayload support to ARM64 platforms
......................................................................
Patch Set 6:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58054/comment/48df738b_a922077b
PS6, Line 12:
> Tested on what device?
MT8195
--
To view, visit https://review.coreboot.org/c/coreboot/+/58054
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I10777a341d46240b91ceeeb1be26c33a0c5db839
Gerrit-Change-Number: 58054
Gerrit-PatchSet: 6
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:32:34 +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: Rex-BC Chen, Paul Menzel.
Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58055 )
Change subject: util/cbfstool: Add support for ARM64 UefiPayload
......................................................................
Patch Set 8:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58055/comment/6b66f40b_9c87db38
PS8, Line 10: ARM SystemReady
> use UEFI + ACPI + SMBIOS on firmware […]
Thanks for answering Paul's question.
--
To view, visit https://review.coreboot.org/c/coreboot/+/58055
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4c0c6e263bd2f518a62ff9db44d72dd31086756a
Gerrit-Change-Number: 58055
Gerrit-PatchSet: 8
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Julius Werner <jwerner(a)chromium.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:30:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Rex-BC Chen, Paul Menzel.
Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58055 )
Change subject: util/cbfstool: Add support for ARM64 UefiPayload
......................................................................
Patch Set 8: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/58055
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4c0c6e263bd2f518a62ff9db44d72dd31086756a
Gerrit-Change-Number: 58055
Gerrit-PatchSet: 8
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Julius Werner <jwerner(a)chromium.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:28:29 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Martin Roth, Rex-BC Chen, Julius Werner, Marty E. Plummer.
Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58054 )
Change subject: payloads: Add UefiPayload support to ARM64 platforms
......................................................................
Patch Set 6: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/58054
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I10777a341d46240b91ceeeb1be26c33a0c5db839
Gerrit-Change-Number: 58054
Gerrit-PatchSet: 6
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:27:03 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Paul Menzel, Julius Werner, Bob Moragues.
Sheng-Liang Pan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58033 )
Change subject: mb/google/trogdor: Add new vaviant quackingstick
......................................................................
Patch Set 3:
(1 comment)
File src/mainboard/google/trogdor/Kconfig:
https://review.coreboot.org/c/coreboot/+/58033/comment/012078ed_1e9aed9e
PS2, Line 15: config TROGDOR_HAS_MIPI_PANEL
> Wait a second... according to b/193585167 this device has a MIPI panel.
Yes...
add MIPI Panel for quackingstick.
--
To view, visit https://review.coreboot.org/c/coreboot/+/58033
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8299ddda14eb82103f17f8464a14992aa757afa6
Gerrit-Change-Number: 58033
Gerrit-PatchSet: 3
Gerrit-Owner: Sheng-Liang Pan <sheng-liang.pan(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: Bob Moragues <moragues(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Sheng-Liang Pan <sheng-liang.pan(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Bob Moragues <moragues(a)chromium.org>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:20:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Julius Werner <jwerner(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Martin Roth, Stefan Reinauer, Julius Werner, Marty E. Plummer.
Rex-BC Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58054 )
Change subject: payloads: Add UefiPayload support to ARM64 platforms
......................................................................
Patch Set 6:
(1 comment)
Patchset:
PS6:
> Seems to be incomplete? Along with CB:58055, attempting to compile coreboot with a tianocore payload […]
Do you mean you build fail for these two patches?
Do you have aby logs for this?
--
To view, visit https://review.coreboot.org/c/coreboot/+/58054
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I10777a341d46240b91ceeeb1be26c33a0c5db839
Gerrit-Change-Number: 58054
Gerrit-PatchSet: 6
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-Comment-Date: Mon, 04 Oct 2021 02:16:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Marty E. Plummer <hanetzer(a)startmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Paul Menzel.
Rex-BC Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58055 )
Change subject: util/cbfstool: Add support for ARM64 UefiPayload
......................................................................
Patch Set 8:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58055/comment/c88dfd03_57b82018
PS8, Line 10: ARM SystemReady
> What is ARM SystemReady?
use UEFI + ACPI + SMBIOS on firmware
https://developer.arm.com/architectures/system-architectures/arm-systemready
--
To view, visit https://review.coreboot.org/c/coreboot/+/58055
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4c0c6e263bd2f518a62ff9db44d72dd31086756a
Gerrit-Change-Number: 58055
Gerrit-PatchSet: 8
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Julius Werner <jwerner(a)chromium.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Mon, 04 Oct 2021 01:55:28 +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: Furquan Shaikh, Martin Roth, Bhanu Prakash Maiya, Paul Menzel, Rob Barnes, Eric Peers, Karthik Ramasubramanian.
Kevin Chiu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57976 )
Change subject: mb/google/guybrush/var/nipperkin: config eSPI alert as in-band
......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/google/guybrush/variants/nipperkin/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/57976/comment/43654789_bcde4f33
PS2, Line 131: register "common_config.espi_config.alert_pin" = "ESPI_ALERT_PIN_IN_BAND"
> We observed eSPI bus errors during S0ix resume when using in-band alerts b/197965075. […]
hi Rob,
can't access b/197965075, could you please loop ODM group in? thank you!
--
To view, visit https://review.coreboot.org/c/coreboot/+/57976
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I18d38fe504bd9f2069b9977d5a35729691f672d1
Gerrit-Change-Number: 57976
Gerrit-PatchSet: 2
Gerrit-Owner: Kevin Chiu <kevin.chiu.17802(a)gmail.com>
Gerrit-Reviewer: Bhanu Prakash Maiya <bhanumaiya(a)google.com>
Gerrit-Reviewer: Eric Peers <epeers(a)google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Rob Barnes <robbarnes(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Bhanu Prakash Maiya <bhanumaiya(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Rob Barnes <robbarnes(a)google.com>
Gerrit-Attention: Eric Peers <epeers(a)google.com>
Gerrit-Attention: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Comment-Date: Mon, 04 Oct 2021 01:26:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Rob Barnes <robbarnes(a)google.com>
Comment-In-Reply-To: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-MessageType: comment