Attention is currently required from: Tristan Corrick, Angel Pons, Alexander Couzens, Evgeny Zinoviev, Patrick Rudolph.
Hello build bot (Jenkins), Arthur Heymans, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59316
to look at the new patch set (#2).
Change subject: cpu/intel/model_206ax: Use static.c exposed lapic 0
......................................................................
cpu/intel/model_206ax: Use static.c exposed lapic 0
Change-Id: Id3b1c4ca027e2905535e137691c3e3e60417dbf3
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/cpu/intel/model_206ax/acpi.c
M src/cpu/intel/model_206ax/chip.h
M src/cpu/intel/model_206ax/model_206ax_init.c
M src/mainboard/apple/macbookair4_2/devicetree.cb
M src/mainboard/asrock/b75pro3-m/devicetree.cb
M src/mainboard/asus/h61-series/devicetree.cb
M src/mainboard/asus/maximus_iv_gene-z/devicetree.cb
M src/mainboard/asus/p8x7x-series/devicetree.cb
M src/mainboard/biostar/th61-itx/devicetree.cb
M src/mainboard/compulab/intense_pc/devicetree.cb
M src/mainboard/dell/optiplex_9010/devicetree.cb
M src/mainboard/gigabyte/ga-b75m-d3h/devicetree.cb
M src/mainboard/gigabyte/ga-h61m-series/devicetree.cb
M src/mainboard/google/butterfly/devicetree.cb
M src/mainboard/google/link/devicetree.cb
M src/mainboard/google/parrot/devicetree.cb
M src/mainboard/google/stout/devicetree.cb
M src/mainboard/hp/compaq_8200_elite_sff/devicetree.cb
M src/mainboard/hp/snb_ivb_laptops/devicetree.cb
M src/mainboard/hp/z220_sff_workstation/devicetree.cb
M src/mainboard/intel/dcp847ske/devicetree.cb
M src/mainboard/intel/emeraldlake2/devicetree.cb
M src/mainboard/kontron/ktqm77/devicetree.cb
M src/mainboard/lenovo/l520/devicetree.cb
M src/mainboard/lenovo/s230u/devicetree.cb
M src/mainboard/lenovo/t420/devicetree.cb
M src/mainboard/lenovo/t420s/devicetree.cb
M src/mainboard/lenovo/t430/devicetree.cb
M src/mainboard/lenovo/t430s/devicetree.cb
M src/mainboard/lenovo/t520/devicetree.cb
M src/mainboard/lenovo/t530/devicetree.cb
M src/mainboard/lenovo/x131e/devicetree.cb
M src/mainboard/lenovo/x1_carbon_gen1/devicetree.cb
M src/mainboard/lenovo/x220/devicetree.cb
M src/mainboard/lenovo/x230/devicetree.cb
M src/mainboard/msi/ms7707/devicetree.cb
M src/mainboard/roda/rv11/variants/rv11/devicetree.cb
M src/mainboard/roda/rv11/variants/rw11/devicetree.cb
M src/mainboard/samsung/lumpy/devicetree.cb
M src/mainboard/samsung/stumpy/devicetree.cb
M src/mainboard/sapphire/pureplatinumh61/devicetree.cb
M src/mainboard/supermicro/x9sae/devicetree.cb
M src/mainboard/supermicro/x9scl/devicetree.cb
43 files changed, 2 insertions(+), 61 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/16/59316/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/59316
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id3b1c4ca027e2905535e137691c3e3e60417dbf3
Gerrit-Change-Number: 59316
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Evgeny Zinoviev <me(a)ch1p.io>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tristan Corrick <tristan(a)corrick.kiwi>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Tristan Corrick <tristan(a)corrick.kiwi>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Attention: Evgeny Zinoviev <me(a)ch1p.io>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Patrick Rudolph.
Hello build bot (Jenkins), Arthur Heymans, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59315
to look at the new patch set (#2).
Change subject: cpu/intel/model_2065x: Use static.c exposed lapic 0
......................................................................
cpu/intel/model_2065x: Use static.c exposed lapic 0
It looks like none of the devicetree were featuring a lapic 0xacac,
nor was tcc_offset ever set, so this remains a NOP.
Change-Id: I296631511b0e31b0ed43ca8193552483bdab4482
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/cpu/intel/model_2065x/chip.h
M src/cpu/intel/model_2065x/model_2065x_init.c
2 files changed, 1 insertion(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/59315/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/59315
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I296631511b0e31b0ed43ca8193552483bdab4482
Gerrit-Change-Number: 59315
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Tristan Corrick, Angel Pons, Alexander Couzens, Patrick Rudolph.
Hello build bot (Jenkins), Arthur Heymans, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59314
to look at the new patch set (#2).
Change subject: cpu/haswell/*.c: Use static.c exposed lapic 0
......................................................................
cpu/haswell/*.c: Use static.c exposed lapic 0
Change-Id: Ic449b2df8036e8c02b5559cca6b2e7479a70a786
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/cpu/intel/haswell/acpi.c
M src/cpu/intel/haswell/chip.h
M src/cpu/intel/haswell/haswell_init.c
M src/mainboard/asrock/b85m_pro4/devicetree.cb
M src/mainboard/asrock/h81m-hds/devicetree.cb
M src/mainboard/google/auron/devicetree.cb
M src/mainboard/google/auron/variants/buddy/overridetree.cb
M src/mainboard/google/auron/variants/samus/overridetree.cb
M src/mainboard/google/beltino/devicetree.cb
M src/mainboard/google/jecht/devicetree.cb
M src/mainboard/google/slippy/devicetree.cb
M src/mainboard/hp/folio_9480m/devicetree.cb
M src/mainboard/intel/baskingridge/devicetree.cb
M src/mainboard/intel/wtm2/devicetree.cb
M src/mainboard/lenovo/t440p/devicetree.cb
M src/mainboard/lenovo/w541/devicetree.cb
M src/mainboard/msi/h81m-p33/devicetree.cb
M src/mainboard/purism/librem_bdw/devicetree.cb
M src/mainboard/supermicro/x10slm-f/devicetree.cb
19 files changed, 4 insertions(+), 43 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/59314/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/59314
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic449b2df8036e8c02b5559cca6b2e7479a70a786
Gerrit-Change-Number: 59314
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tristan Corrick <tristan(a)corrick.kiwi>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Tristan Corrick <tristan(a)corrick.kiwi>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Martin Roth, Angel Pons.
Hello build bot (Jenkins), Martin Roth, Patrick Georgi, Angel Pons, Arthur Heymans,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59313
to look at the new patch set (#2).
Change subject: util/sconfig: Expose apic devices
......................................................................
util/sconfig: Expose apic devices
Expose apic devices in static.c such that fake 0xACAC lapics are not
needed in most cases.
Change-Id: I800f089998f4a9be2216ed997cec5cedeb63f3db
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/include/device/device.h
M util/sconfig/main.c
2 files changed, 16 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/59313/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/59313
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I800f089998f4a9be2216ed997cec5cedeb63f3db
Gerrit-Change-Number: 59313
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Дмитрий Понаморев, Kyösti Mälkki.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/blobs/+/57196 )
Change subject: 3rdparty/blobs/mainboard/teleplatforms/D4E4S16P8: Create new folder for teleplatforms/D4E4S16P8 CRB.
......................................................................
Patch Set 4:
(5 comments)
Commit Message:
https://review.coreboot.org/c/blobs/+/57196/comment/dbbd881d_ae86ab78
PS4, Line 7: 3rdparty/blobs/mainboard/teleplatforms/D4E4S16P8
Maybe:
> mb/teleplatforms/D4E4S168: Add blobs
https://review.coreboot.org/c/blobs/+/57196/comment/4097ce7a_bde52a83
PS4, Line 7: 3rdparty/blobs/mainboard/teleplatforms/D4E4S16P8: Create new folder for teleplatforms/D4E4S16P8 CRB.
No dot/period at the end of commit message summaries please [1].
[1]: https://chris.beams.io/posts/git-commit/https://review.coreboot.org/c/blobs/+/57196/comment/ba98a2bb_01c65aac
PS4, Line 9: compile
build
https://review.coreboot.org/c/blobs/+/57196/comment/2b5dc6b3_467db06e
PS4, Line 11:
Please mention somewhere, where you got the permissions to distribute the blobs from. For example, what is `pci1a03,2000.rom` (also what version), and what license does it have?
Patchset:
PS4:
Maybe also list the versions of the other components (ME, FSP, …).
--
To view, visit https://review.coreboot.org/c/blobs/+/57196
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: blobs
Gerrit-Branch: master
Gerrit-Change-Id: I71de576aa283fe6738138ad023d70c282045f5cc
Gerrit-Change-Number: 57196
Gerrit-PatchSet: 4
Gerrit-Owner: Дмитрий Понаморев <dponamorev(a)gmail.com>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Дмитрий Понаморев <dponamorev(a)gmail.com>
Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Comment-Date: Wed, 17 Nov 2021 09:15:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Martin Roth, Angel Pons, Arthur Heymans.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59313 )
Change subject: util/sconfig: Expose apic devices
......................................................................
Patch Set 1:
(1 comment)
File src/include/device/device.h:
https://review.coreboot.org/c/coreboot/+/59313/comment/b7be7d87_8a25b4c8
PS1, Line 394: __apic_0000_0
> This doesn't look like a config aka chip_info
heh I misread the line below. thank you
--
To view, visit https://review.coreboot.org/c/coreboot/+/59313
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I800f089998f4a9be2216ed997cec5cedeb63f3db
Gerrit-Change-Number: 59313
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Comment-Date: Wed, 17 Nov 2021 09:04:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Bao Zheng, Martin Roth, Zheng Bao.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57131 )
Change subject: amdfwtool: Add support for AMD's BIOS recovery feature
......................................................................
Patch Set 42:
(1 comment)
File util/amdfwtool/amdfwtool.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133593):
https://review.coreboot.org/c/coreboot/+/57131/comment/79dae5f4_55522aaa
PS42, Line 1892: btl_entry = search_entry(pspdir2, AMD_FW_PSP_BOOTLOADER);
line over 96 characters
--
To view, visit https://review.coreboot.org/c/coreboot/+/57131
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2671b95fe089aafdaf61b55bc9d2e8dcf6a66dbc
Gerrit-Change-Number: 57131
Gerrit-PatchSet: 42
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Bao Zheng <zheng.bao(a)amd.corp-partner.google.com>
Gerrit-CC: Martin Roth <martinroth(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Zheng Bao
Gerrit-Comment-Date: Wed, 17 Nov 2021 08:51:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment