build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67365 )
Change subject: Update arm-trusted-firmware submodule to upstream master
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158092):
https://review.coreboot.org/c/coreboot/+/67365/comment/94ed246f_a61f4bde
PS1, Line 9: Updating from commit id e0a6a512b:
Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158092):
https://review.coreboot.org/c/coreboot/+/67365/comment/b769d4ad_fde958dc
PS1, Line 12: to commit id 7805999e6:
Possible unwrapped commit description (prefer a maximum 72 chars per line)
--
To view, visit https://review.coreboot.org/c/coreboot/+/67365
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I19e4cb1d423086796040834ebce9262d39935e90
Gerrit-Change-Number: 67365
Gerrit-PatchSet: 1
Gerrit-Owner: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 06 Sep 2022 06:29:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Rex-BC Chen, Yidi Lin.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67357 )
Change subject: mb/google/geralt: Pass reset gpio parameter to BL31
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/67357/comment/2e24719b_46361cfe
PS1, Line 9: To support gpio reset SoC, we need to pass the reset gpio parameter to
: BL31.
Pass the reset gpio parameter to BL31 to support SoC reset.
--
To view, visit https://review.coreboot.org/c/coreboot/+/67357
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifdfbd6bd82f64b084f6349cb617443053c89a3f1
Gerrit-Change-Number: 67357
Gerrit-PatchSet: 1
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Yidi Lin <yidilin(a)google.com>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.com>
Gerrit-Attention: Yidi Lin <yidilin(a)google.com>
Gerrit-Comment-Date: Tue, 06 Sep 2022 06:22:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Hung-Te Lin, Rex-BC Chen, Yidi Lin.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67355 )
Change subject: soc/mediatek/mt8188: enable mfgpll properly and fix SPMI muxes
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/67355/comment/c422c9d5_c2a6de7b
PS1, Line 9: We failed to bringup GPU in kernel due to the wrong pll setting.
Some of the pll settings are incorrect, which cause problems in GPU after booting into kernel.
File src/soc/mediatek/mt8188/pll.c:
https://review.coreboot.org/c/coreboot/+/67355/comment/6a9242f5_5cb80c55
PS1, Line 526: setbits32
Use `clrsetbits32`.
--
To view, visit https://review.coreboot.org/c/coreboot/+/67355
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I971109a5f72e3307899daaf5a5f26022124b559b
Gerrit-Change-Number: 67355
Gerrit-PatchSet: 1
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Yidi Lin <yidilin(a)google.com>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.com>
Gerrit-Attention: Yidi Lin <yidilin(a)google.com>
Gerrit-Comment-Date: Tue, 06 Sep 2022 06:20:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Martin L Roth, Patrick Georgi, Maxim Polyakov, Paul Menzel, Angel Pons, ron minnich.
David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64724 )
Change subject: intelp2m: Add Go Managing Dependencies System support
......................................................................
Patch Set 7: Code-Review+2
(1 comment)
Patchset:
PS7:
I was able to apply this patch to a personal repo, modify the URLs, and import from another project and it seemed to work fine. Thanks!
--
To view, visit https://review.coreboot.org/c/coreboot/+/64724
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I12efae31227129b8c884af10fb233f398c4094e7
Gerrit-Change-Number: 64724
Gerrit-PatchSet: 7
Gerrit-Owner: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich(a)gmail.com>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: ron minnich <rminnich(a)gmail.com>
Gerrit-Comment-Date: Tue, 06 Sep 2022 05:23:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67358 )
Change subject: Update arm-trusted-firmware submodule to upstream master
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158090):
https://review.coreboot.org/c/coreboot/+/67358/comment/40809883_74f9cca9
PS1, Line 9: Updating from commit id e0a6a512b:
Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158090):
https://review.coreboot.org/c/coreboot/+/67358/comment/a5fcee1a_471e47bc
PS1, Line 12: to commit id 7805999e6:
Possible unwrapped commit description (prefer a maximum 72 chars per line)
--
To view, visit https://review.coreboot.org/c/coreboot/+/67358
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I981956fbdcbcfa4ce185652478b9bb30d40f5686
Gerrit-Change-Number: 67358
Gerrit-PatchSet: 1
Gerrit-Owner: Yidi Lin <yidilin(a)google.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 06 Sep 2022 04:20:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment