coreboot-gerrit September 2021

coreboot-gerrit@coreboot.org

1 participants
6530 discussions

Change in coreboot[master]: [TESTME] mb/pcengines/apu2/romstage: use proper GPIO configuration API
by Michał Żygowski (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Felix Held, Piotr Król. Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56784 ) Change subject: [TESTME] mb/pcengines/apu2/romstage: use proper GPIO configuration API ...................................................................... Patch Set 2: Code-Review+2 (1 comment) Patchset: PS2: Works well, no issues detected by our testing suites. Felix you may add `Tested-by: Michał Kopeć <michal.kopec(a)3mdeb.com>` to the commit mesage -- To view, visit https://review.coreboot.org/c/coreboot/+/56784 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If121941c8a6ba88913653192740997aeef426548 Gerrit-Change-Number: 56784 Gerrit-PatchSet: 2 Gerrit-Owner: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com> Gerrit-Reviewer: Piotr Król <piotr.krol(a)3mdeb.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Attention: Piotr Król <piotr.krol(a)3mdeb.com> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:37:49 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: mb/google/dedede/var/gooey: Configure I2C times for I2C devices
by Henry Sun (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Stanley Wu, Paul Fagerburg. Henry Sun has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57297 ) Change subject: mb/google/dedede/var/gooey: Configure I2C times for I2C devices ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > CR+2 approved. Thank you, please wait for next weeks bunch merge then, should be with this CL in. -- To view, visit https://review.coreboot.org/c/coreboot/+/57297 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ic5d5660181f36f161ae04cbf5003f6d7ad4bc16f Gerrit-Change-Number: 57297 Gerrit-PatchSet: 1 Gerrit-Owner: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com> Gerrit-Reviewer: Henry Sun <henrysun(a)google.com> Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com> Gerrit-Reviewer: Paul Fagerburg <pfagerburg(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Jerry2 Huang <jerry2.huang(a)lcfc.corp-partner.google.com> Gerrit-CC: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com> Gerrit-CC: Kevin Chang <kevin.chang(a)lcfc.corp-partner.google.com> Gerrit-CC: Sunshine Chao <sunshine.chao(a)lcfc.corp-partner.google.com> Gerrit-Attention: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com> Gerrit-Attention: Paul Fagerburg <pfagerburg(a)chromium.org> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:35:00 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Henry Sun <henrysun(a)google.com> Comment-In-Reply-To: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: src/mainboard/herobrine: Load GSI FW in ramstage
by Paul Menzel (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Shelley Chen, Ravi kumar, mturney mturney, Julius Werner. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55964 ) Change subject: src/mainboard/herobrine: Load GSI FW in ramstage ...................................................................... Patch Set 17: (1 comment) File src/mainboard/google/herobrine/mainboard.c: https://review.coreboot.org/c/coreboot/+/55964/comment/340115a9_8b83cfed PS17, Line 25: qupv3_se_fw_load_and_init(QUPV3_0_SE1, SE_PROTOCOL_I2C, GSI); /* APPS I2C */ How much time does this take? Please add that to the commit message. -- To view, visit https://review.coreboot.org/c/coreboot/+/55964 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I3d9caa0921fcf9ad67f1071cdf769a99fb6d1a30 Gerrit-Change-Number: 55964 Gerrit-PatchSet: 17 Gerrit-Owner: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org> Gerrit-Reviewer: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: mturney mturney <mturney(a)codeaurora.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Rajesh Patil <rajpat(a)qualcomm.corp-partner.google.com> Gerrit-CC: Ravi Kumar Bokka <c_rbokka(a)qualcomm.corp-partner.google.com> Gerrit-CC: Roja Rani Yarubandi <c_rojay(a)qualcomm.corp-partner.google.com> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Attention: mturney mturney <mturney(a)codeaurora.org> Gerrit-Attention: Julius Werner <jwerner(a)chromium.org> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:28:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: mb/google/dedede: Create beetley variant
by Paul Menzel (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Zhi7 Li. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57358 ) Change subject: mb/google/dedede: Create beetley variant ...................................................................... Patch Set 1: (1 comment) Commit Message: https://review.coreboot.org/c/coreboot/+/57358/comment/bdd85042_30a9fa04 PS1, Line 12: (Auto-Generated by create_coreboot_variant.sh version 4.5.0). Please move the dot/period before the ). -- To view, visit https://review.coreboot.org/c/coreboot/+/57358 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I3d52be55270361406a67f91707b71dff5c725f35 Gerrit-Change-Number: 57358 Gerrit-PatchSet: 1 Gerrit-Owner: Zhi7 Li <lizhi7(a)huaqin.corp-partner.google.com> Gerrit-Reviewer: Weimin Wu <wuweimin(a)huaqin.corp-partner.google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Krystal Han <hanlijing(a)huaqin.corp-partner.google.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Zhi7 Li <lizhi7(a)huaqin.corp-partner.google.com> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:27:44 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: mb/google/brya/variants/gimble: Enable SaGv support
by Paul Menzel (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: YH Lin, Tim Wawrzynczak, Kane Chen, Mark Hsieh, Felix Held. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57357 ) Change subject: mb/google/brya/variants/gimble: Enable SaGv support ...................................................................... Patch Set 2: (1 comment) Commit Message: https://review.coreboot.org/c/coreboot/+/57357/comment/bcb440e9_0a7ea40c PS2, Line 9: This patch enables SaGv support for gimble. Why can it be enabled? -- To view, visit https://review.coreboot.org/c/coreboot/+/57357 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I29887418827614afb10558c6958c9c5e9667079e Gerrit-Change-Number: 57357 Gerrit-PatchSet: 2 Gerrit-Owner: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Kane Chen <kane.chen(a)intel.com> Gerrit-Reviewer: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com> Gerrit-Reviewer: Marx Wang <marx.wang(a)intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: YH Lin <yueherngl(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Anfernee Chen <anfernee_chen(a)wistron.corp-partner.google.com> Gerrit-CC: Ariel Fang <ariel_fang(a)wistron.corp-partner.google.com> Gerrit-CC: Casper Chang <casper_chang(a)wistron.corp-partner.google.com> Gerrit-CC: Malik Hsu <malik_hsu(a)wistron.corp-partner.google.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Scott Chao <scott_chao(a)wistron.corp-partner.google.com> Gerrit-CC: Terry Chen <terry_chen(a)wistron.corp-partner.google.com> Gerrit-CC: Will Tsai <will_tsai(a)wistron.corp-partner.google.com> Gerrit-Attention: YH Lin <yueherngl(a)google.com> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Kane Chen <kane.chen(a)intel.com> Gerrit-Attention: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com> Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:27:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: src/mainboard/trogdor: Initialize SPI FW for EC and TPM
by Paul Menzel (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Shelley Chen, Ravi kumar. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56398 ) Change subject: src/mainboard/trogdor: Initialize SPI FW for EC and TPM ...................................................................... Patch Set 10: (1 comment) File src/mainboard/google/trogdor/bootblock.c: https://review.coreboot.org/c/coreboot/+/56398/comment/d36ab722_a945a0b4 PS10, Line 10: 1010 * KHz Please mention where the frequency is documented in the commit message. -- To view, visit https://review.coreboot.org/c/coreboot/+/56398 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifb89e36517f6a32b8b5881b89112dac4a3f292b0 Gerrit-Change-Number: 56398 Gerrit-PatchSet: 10 Gerrit-Owner: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Ravi Kumar Bokka <c_rbokka(a)qualcomm.corp-partner.google.com> Gerrit-CC: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Ravi kumar <rbokka(a)codeaurora.org> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:26:49 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: mb/google/dedede/var/gooey: Add MT53E512M32D1NP-046 as supported mem ...
by Paul Menzel (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Henry Sun, Stanley Wu, Karthik Ramasubramanian. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57310 ) Change subject: mb/google/dedede/var/gooey: Add MT53E512M32D1NP-046 as supported mem module ...................................................................... Patch Set 2: Code-Review+1 (1 comment) Commit Message: https://review.coreboot.org/c/coreboot/+/57310/comment/ca8da12c_a02974f4 PS1, Line 11: MT53E512M32D1NP-046 WT:B > From Micron AE feedback as below: […] Could you add that to the commit message please? -- To view, visit https://review.coreboot.org/c/coreboot/+/57310 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I7b83126a2bf98bb9d0ca05d397c288e0d99ed781 Gerrit-Change-Number: 57310 Gerrit-PatchSet: 2 Gerrit-Owner: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com> Gerrit-Reviewer: Henry Sun <henrysun(a)google.com> Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com> Gerrit-Reviewer: Paul Fagerburg <pfagerburg(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Jerry2 Huang <jerry2.huang(a)lcfc.corp-partner.google.com> Gerrit-CC: Sunshine Chao <sunshine.chao(a)lcfc.corp-partner.google.com> Gerrit-Attention: Henry Sun <henrysun(a)google.com> Gerrit-Attention: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com> Gerrit-Attention: Karthik Ramasubramanian <kramasub(a)google.com> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:25:35 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org> Comment-In-Reply-To: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: Rename MMCONF Kconfigs to ECAM_MMCONF
by Furquan Shaikh (Code Review) 03 Sep '21

03 Sep '21

1 0

Change in coreboot[master]: Rename MMCONF Kconfigs to ECAM_MMCONF
by Furquan Shaikh (Code Review) 03 Sep '21

03 Sep '21

Attention is currently required from: Raul Rangel, Mariusz Szafrański, Jonathan Zhang, Paul Menzel, Stefan Reinauer, Kyösti Mälkki, Andrey Petrov, Patrick Rudolph, Nico Huber, Anjaneya "Reddy" Chagam, Johnny Lin, Suresh Bellampalli, Morgan Jang, Michal Motyl, Alexander Couzens, Felix Held, Shelley Chen, Angel Pons, Lance Zhao, Jason Glenesk, Martin Roth, Damien Zammit, Lee Leahy, Marshall Dawson, Tim Wawrzynczak, Vanessa Eusebio, Huang Jin. Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57333 ) Change subject: Rename MMCONF Kconfigs to ECAM_MMCONF ...................................................................... Patch Set 5: (13 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/57333/comment/e3dc103b_c5612532 PS3, Line 11: which applies only to x86 platforms Actually, this applies to platforms that support ACPI. https://review.coreboot.org/c/coreboot/+/57333/comment/652b14d9_31b0d602 PS3, Line 11: This can be confusing as : other platforms, such as ARM, use a different way of mapping the pci : config space to memory. > My understanding so far (for QC IIUC) is that there is a single 4K window in MMIO space, and some ot […] That sounds right for QC platform. Other platforms could implement this in other ways too. https://review.coreboot.org/c/coreboot/+/57333/comment/3fb8efc2_9dc97bca PS3, Line 16: NO_ECAM_MMCONF_SUPPORT Reading the PCI firmware specification again, ECAM and MMCONF actually go hand in hand. So, Commit Message: https://review.coreboot.org/c/coreboot/+/57333/comment/ceb292de_5a3d19da PS4, Line 9: the MMCONF Kconfigs only use the Enhanced Configuration : Access mechanism (ECAM) method Reading following two docs again: * PCI base express specification * PCI firmware specification This is what I understand: ECAM and MMCONFIG go hand in hand. PCI base express specification mentions that platforms that support ACPI must implement Enhanced configuration access mechanism (ECAM). This allows PCI configuration space access using memory primitives instead of I/O-based primitives (CF8/CFC). In order to report the memory mapped configuration (MMCONFIG) space base address, MCFG table is used in ACPI. Given that, I think we need to set up things such that: There are two types of PCI ops supported: * PCI_IO_OPS * PCI_MMIO_OPS For PCI_MMIO_OPS, there will be PCI_ECAM which will be selected by default if PCI and ACPI are both selected for a platform. If ACPI is not selected, then the platform can either choose to select PCI_ECAM or it can provide its own access mechanisms. I think the current MMCONF Kconfigs are really a mix of MMIO ops and ECAM. So, we will have to inspect and create two different Kconfigs i.e. PCI_MMIO_OPS and PCI_ECAM and accordingly update the current configs. PCI_MMIO_OPS must be selected by default if PCI is selected unless a platform selects PCI_NO_MMIO_OPS. https://review.coreboot.org/c/coreboot/+/57333/comment/a6cb1a94_48b06689 PS4, Line 11: which applies only to x86 platforms Actually, this applies to platforms that support ACPI. So, if there are non-x86 platforms that support ACPI, they will support ECAM. https://review.coreboot.org/c/coreboot/+/57333/comment/fae77218_acefbd93 PS4, Line 16: NO_ECAM_MMCONF_SUPPORT I think this will have to be renamed to `PCI_NO_MMIO_OPS` https://review.coreboot.org/c/coreboot/+/57333/comment/be0532ce_e5c93a8b PS4, Line 17: ECAM_MMCONF_SUPPORT This will really be either `PCI_ECAM` or `PCI_MMIO_OPS` depending upon how it is used. https://review.coreboot.org/c/coreboot/+/57333/comment/41e121a3_ffd3a241 PS4, Line 18: ECAM_MMCONF_BASE_ADDRESS I think `MMCONF_BASE_ADDRESS` is fine here because that is how the PCI spec uses it. Probably this can be renamed to PCI_MMCONF_BASE_ADDRESS and add dependency on `PCI_ECAM`. https://review.coreboot.org/c/coreboot/+/57333/comment/a74c46cc_cb899b24 PS4, Line 19: MMCONF_BUS_NUMBER --> ECAM_MMCONF_BUS_NUMBER : MMCONF_LENGTH --> ECAM_MMCONF_LENGTH Same for these two Kconfigs as `MMCONF_BASE_ADDRESS`. File src/arch/x86/include/arch/pci_io_cfg.h: https://review.coreboot.org/c/coreboot/+/57333/comment/e50876be_5806b636 PS4, Line 72: ECAM_MMCONF_SUPPORT This should be `!CONFIG(PCI_MMIO_OPS)` File src/device/Kconfig: https://review.coreboot.org/c/coreboot/+/57333/comment/8eda7e1c_0a70d7ce PS4, Line 502: NO_ECAM_MMCONF_SUPPORT Posted a comment on the commit message about the Kconfigs. File src/northbridge/intel/e7505/Kconfig: https://review.coreboot.org/c/coreboot/+/57333/comment/711d9d6d_ab3d04c9 PS4, Line 10: NO_ECAM_MMCONF_SUPPORT This will have to be changed to PCI_NO_MMIO_OPS File src/soc/intel/quark/Kconfig: https://review.coreboot.org/c/coreboot/+/57333/comment/63335c3b_310ce292 PS4, Line 13: NO_ECAM_MMCONF_SUPPORT This should be PCI_NO_MMIO_OPS -- To view, visit https://review.coreboot.org/c/coreboot/+/57333 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1e196a1ed52d131a71f00cba1d93a23e54aca3e2 Gerrit-Change-Number: 57333 Gerrit-PatchSet: 5 Gerrit-Owner: Shelley Chen <shchen(a)google.com> Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu> Gerrit-Reviewer: Andrey Petrov <andrey.petrov(a)gmail.com> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Anjaneya "Reddy" Chagam <anjaneya.chagam(a)intel.com> Gerrit-Reviewer: Damien Zammit Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com> Gerrit-Reviewer: Huang Jin <huang.jin(a)intel.com> Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com> Gerrit-Reviewer: Johnny Lin <Johnny_Lin(a)wiwynn.com> Gerrit-Reviewer: Jonathan Zhang <jonzhang(a)fb.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Lance Zhao Gerrit-Reviewer: Lee Leahy <leroy.p.leahy(a)intel.com> Gerrit-Reviewer: Mariusz Szafrański <mariuszx.szafranski(a)intel.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl(a)intel.com> Gerrit-Reviewer: Morgan Jang <Morgan_Jang(a)wiwynn.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: Suresh Bellampalli <suresh.bellampalli(a)intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski(a)intel.com> Gerrit-Attention: Jonathan Zhang <jonzhang(a)fb.com> Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Attention: Andrey Petrov <andrey.petrov(a)gmail.com> Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Attention: Nico Huber <nico.h(a)gmx.de> Gerrit-Attention: Anjaneya "Reddy" Chagam <anjaneya.chagam(a)intel.com> Gerrit-Attention: Johnny Lin <Johnny_Lin(a)wiwynn.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli(a)intel.com> Gerrit-Attention: Morgan Jang <Morgan_Jang(a)wiwynn.com> Gerrit-Attention: Michal Motyl <michalx.motyl(a)intel.com> Gerrit-Attention: Alexander Couzens <lynxis(a)fe80.eu> Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Attention: Shelley Chen <shchen(a)google.com> Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: Lance Zhao Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com> Gerrit-Attention: Martin Roth <martinroth(a)google.com> Gerrit-Attention: Damien Zammit Gerrit-Attention: Lee Leahy <leroy.p.leahy(a)intel.com> Gerrit-Attention: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio(a)intel.com> Gerrit-Attention: Huang Jin <huang.jin(a)intel.com> Gerrit-Comment-Date: Fri, 03 Sep 2021 08:22:21 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: Rename MMCONF Kconfigs to ECAM_MMCONF
by Shelley Chen (Code Review) 03 Sep '21

03 Sep '21

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit September 2021