Attention is currently required from: Arthur Heymans, Benjamin Doron, Felix Held, Patrick Rudolph.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79160?usp=email )
Change subject: Documentation: Improve x86_64
......................................................................
Patch Set 2: Code-Review+1
(3 comments)
Patchset:
PS2:
Thanks again for working on this documentation.
File Documentation/arch/x86/x86_64.md:
https://review.coreboot.org/c/coreboot/+/79160/comment/44c751da_9a598ab2 :
PS1, Line 24:
: The large memory model causes the compiler to emit 64bit addressing
: instructions, which are not only slower, but also increase code size.
> Page tables can be used to provide security benefits, such as by marking memory as non-executable or […]
Sorry, I was suggesting that the response to the question be added into the document. The way it's currently worded leaves one wondering if there's any benefit and why we'd want it.
https://review.coreboot.org/c/coreboot/+/79160/comment/ea1189b0_890b8a9b :
PS1, Line 28: must generate page tables at build time
> I'm not sure I understand? If you're asking if long mode architecturally needs page tables in ROM, n […]
The statement is that the toolchain *must* do this. Maybe rephrase it saying that this is coreboot's implementation and not a requirement.
--
To view, visit https://review.coreboot.org/c/coreboot/+/79160?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia5ba51be629a8c878aad64d3297176457cf8e855
Gerrit-Change-Number: 79160
Gerrit-PatchSet: 2
Gerrit-Owner: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Attention: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Wed, 27 Dec 2023 03:06:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Martin L Roth <gaumless(a)gmail.com>
Comment-In-Reply-To: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Nick Vaccaro, Subrata Banik, Weimin Wu.
Eric Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79730?usp=email )
Change subject: mb/google/nissa/var/anraggar: add hook for WiFi SAR table
......................................................................
Patch Set 5:
(1 comment)
File src/mainboard/google/brya/variants/anraggar/variant.c:
https://review.coreboot.org/c/coreboot/+/79730/comment/b208600e_6a93ad2e :
PS5, Line 7: return "wifi_sar_2.hex";
Not sure do you want to apply use the FW config, but up to you.
--
To view, visit https://review.coreboot.org/c/coreboot/+/79730?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Idb200699bb8c8581b9512ec8ec9442f65f8822b3
Gerrit-Change-Number: 79730
Gerrit-PatchSet: 5
Gerrit-Owner: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eric Lai <ericllai(a)google.com>
Gerrit-Reviewer: Jianeng Ceng <cengjianeng(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Tao Yang <yangtao6(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-Comment-Date: Wed, 27 Dec 2023 02:30:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Nick Vaccaro, Subrata Banik, Weimin Wu.
Eric Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79730?usp=email )
Change subject: mb/google/nissa/var/anraggar: add hook for WiFi SAR table
......................................................................
Patch Set 5: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/79730?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Idb200699bb8c8581b9512ec8ec9442f65f8822b3
Gerrit-Change-Number: 79730
Gerrit-PatchSet: 5
Gerrit-Owner: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eric Lai <ericllai(a)google.com>
Gerrit-Reviewer: Jianeng Ceng <cengjianeng(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Tao Yang <yangtao6(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-Comment-Date: Wed, 27 Dec 2023 02:30:18 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Iru Cai.
Nicholas Chin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79746?usp=email )
Change subject: mainboard: add Dell Latitude E7240
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS1:
> I wrote the same "chip ec/dell/mec5035" block as e6400. […]
Ah, the code for keyboard/touchpad init is done in ramstage so if those ports weren't enabled then the system probably just got stuck waiting for the EC.
--
To view, visit https://review.coreboot.org/c/coreboot/+/79746?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I6933bdbcc8d0bbb85d62657624740266284ac71c
Gerrit-Change-Number: 79746
Gerrit-PatchSet: 2
Gerrit-Owner: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-Comment-Date: Wed, 27 Dec 2023 02:23:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nicholas Chin <nic.c3.14(a)gmail.com>
Comment-In-Reply-To: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Nicholas Chin.
Iru Cai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79746?usp=email )
Change subject: mainboard: add Dell Latitude E7240
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS1:
> Perhaps it didn't reuse CB:40300 because it is abandonded. […]
I wrote the same "chip ec/dell/mec5035" block as e6400. I just noticed that the EC driver also uses 0x910/0x911 ports, but I don't enable these ports after bootblock. I'll try to enable these ports in ramstage and test again.
--
To view, visit https://review.coreboot.org/c/coreboot/+/79746?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I6933bdbcc8d0bbb85d62657624740266284ac71c
Gerrit-Change-Number: 79746
Gerrit-PatchSet: 2
Gerrit-Owner: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Comment-Date: Wed, 27 Dec 2023 02:06:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nicholas Chin <nic.c3.14(a)gmail.com>
Comment-In-Reply-To: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Eric Lai, Nick Vaccaro, Subrata Banik, Weimin Wu.
Jianeng Ceng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79730?usp=email )
Change subject: mb/google/nissa/var/anraggar: add hook for WiFi SAR table
......................................................................
Patch Set 5: Code-Review+1
(2 comments)
File src/mainboard/google/brya/variants/anraggar/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/79730/comment/0d556296_e2c99bf9 :
PS3, Line 8: ramstage-y += variant.c
> Check that files end with a single newline (lint-extended-015-final-newlines): src/mainboard/google/ […]
Done
File src/mainboard/google/brya/variants/anraggar/variant.c:
https://review.coreboot.org/c/coreboot/+/79730/comment/0e23e540_b83ac5fb :
PS2, Line 7: return "wifi_sar_2.hex";
> add tab
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/79730?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Idb200699bb8c8581b9512ec8ec9442f65f8822b3
Gerrit-Change-Number: 79730
Gerrit-PatchSet: 5
Gerrit-Owner: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eric Lai <ericllai(a)google.com>
Gerrit-Reviewer: Jianeng Ceng <cengjianeng(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Tao Yang <yangtao6(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Eric Lai <ericllai(a)google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com>
Gerrit-Comment-Date: Wed, 27 Dec 2023 01:40:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Eric Lai <ericllai(a)google.com>
Gerrit-MessageType: comment
Attention is currently required from: nat ✨.
Michael Büchler has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55232?usp=email )
Change subject: mb/dell: Add OptiPlex 7020/9020 port
......................................................................
Patch Set 31:
(1 comment)
Patchset:
PS31:
> I got this running on my 9020 SFF. […]
Regarding EDK2 with discrete graphics and GOP: this May I was told on IRC that the usual way is not EDK II calling the Option ROM to set up the GOP structure. Instead, coreboot runs the Option ROM (or libgfxinit), passes information about the VESA framebuffer to EDK II, and EDK II creates the GOP structure from that. Use `CONFIG_FRAMEBUFFER_SET_VESA_MODE`. I had an issue that you might also run into: https://github.com/tianocore/edk2/pull/4631
There was also speculation that some fork of EDK II (maybe 9elements) could run the Option ROM for the GOP thing. Or maybe something changed in the meantime - these were just my notes from some months ago. Happy tinkering! (no clue about SATA or ReBAR, sorry)
--
To view, visit https://review.coreboot.org/c/coreboot/+/55232?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ie7c7089f443aef9890711c4412209bceb1f1e96a
Gerrit-Change-Number: 55232
Gerrit-PatchSet: 31
Gerrit-Owner: Máté Kukri <kukri.mate(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Michael Büchler <michael.buechler(a)posteo.net>
Gerrit-Reviewer: Patrick Rudolph <rudolphpatrick05(a)gmail.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Ben Westover <me(a)benthetechguy.net>
Gerrit-CC: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-CC: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-CC: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: nat ✨ <nat(a)nekopon.pl>
Gerrit-Attention: nat ✨ <nat(a)nekopon.pl>
Gerrit-Comment-Date: Wed, 27 Dec 2023 00:24:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: nat ✨ <nat(a)nekopon.pl>
Gerrit-MessageType: comment