Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45131 )
Change subject: lib/Makefile.inc: fail build when SPD would be empty
......................................................................
Patch Set 8:
> Patch Set 8:
>
> > Patch Set 6:
> >
> > > Patch Set 6:
> > >
> > > > Patch Set 6:
> > > >
> > > > Does anyone know which kind of memory (slotted or memory-down) Google Sushi and Google Trondo use?
> > >
> > > Sushi was a test platform, and won't actually be built. But if you need to specify something, then memory-down.
> >
> > Right, thanks for the insight. If Sushi won't actually be built, we might as well drop it then? Treating it like a memory-down platform would require adding an invalid SPD binary to it, which wouldn't work.
>
> I'll look at what's involved besides just removing it from coreboot.
https://review.coreboot.org/c/coreboot/+/45175
--
To view, visit https://review.coreboot.org/c/coreboot/+/45131
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic6db1dbe5fed5f242e408bcad4f36dda1b1fa1b4
Gerrit-Change-Number: 45131
Gerrit-PatchSet: 8
Gerrit-Owner: Michael Niewöhner
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Caveh Jalali <caveh(a)chromium.org>
Gerrit-Reviewer: David Wu <david_wu(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Paul Fagerburg <pfagerburg(a)chromium.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 08 Sep 2020 17:01:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Hello build bot (Jenkins), Nico Huber, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39995
to look at the new patch set (#15).
Change subject: mb/siemens/chili: Add Chili variant
......................................................................
mb/siemens/chili: Add Chili variant
* Update GMA Port_List for Chili variant
* Comment Chili-variant devicetree
* Add option to disable `ethernet1`
* Implement preliminary fan config for Chili
* Slightly adjust fan control of Chili
* Set HWM limits and tune fan config of Chili
* Revise Chili's PCI devices
Mention onboard PCI devices explicitly, disable unused `debug` PCIe
root port and reorder other root ports according to coalescing rules.
* Add bx26rom support for Chili
* Configure EC device enablement
Individual bits are based on observation of the original BIOS. For
the Chili variant, add a Kconfig option to enable the debug USB port.
* Add HDA verbs for Chili
* Move mainboard_init() (GPIO config) to romstage
* Update devicetrees wrt. SlotImplemented
* Update Chili's fan config
* Ignore mic presence on Chili
Change-Id: Ic7a5dccbb0d5b7bceb154fb050cf991254475f7b
Signed-off-by: Nico Huber <nico.huber(a)secunet.com>
Signed-off-by: Felix Singer <felix.singer(a)secunet.com>
---
M src/mainboard/siemens/chili/Kconfig
M src/mainboard/siemens/chili/Kconfig.name
M src/mainboard/siemens/chili/mainboard.c
M src/mainboard/siemens/chili/romstage.c
M src/mainboard/siemens/chili/variant.h
A src/mainboard/siemens/chili/variants/chili/Makefile.inc
A src/mainboard/siemens/chili/variants/chili/board_info.txt
A src/mainboard/siemens/chili/variants/chili/boxer26_eeprom.h
A src/mainboard/siemens/chili/variants/chili/data.vbt
A src/mainboard/siemens/chili/variants/chili/devicetree.cb
A src/mainboard/siemens/chili/variants/chili/gma-mainboard.ads
A src/mainboard/siemens/chili/variants/chili/gpio.c
A src/mainboard/siemens/chili/variants/chili/hda_verb.c
A src/mainboard/siemens/chili/variants/chili/romstage.c
A src/mainboard/siemens/chili/variants/chili/smbios.c
15 files changed, 672 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/39995/15
--
To view, visit https://review.coreboot.org/c/coreboot/+/39995
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic7a5dccbb0d5b7bceb154fb050cf991254475f7b
Gerrit-Change-Number: 39995
Gerrit-PatchSet: 15
Gerrit-Owner: Felix Singer <felixsinger(a)posteo.net>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset