Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31936 )
Change subject: [RFC] util/sconfig: Drop topology from early static tree
......................................................................
Patch Set 7:
- having config data early (bootblock, etc) but remove the bloat from the full device tree. In many of these scenarios we don't need the device itself. We just need the config object.
We could emit references to the config objects, the same as for
device nodes. e.g. like this:
DEVTREE_CONST struct northbridge_intel_gm45_config *const __pci_0_00_0_chip_info = northbridge_intel_gm45_info_1;
Then `__pci_0_00_0->chip_info` would become `__pci_0_00_0_chip_info`
and GC could take care of the device node if it's not referenced
otherwise.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31936
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Icdc63123fb8bcb39a0fc86e1e60b8eebb909223c
Gerrit-Change-Number: 31936
Gerrit-PatchSet: 7
Gerrit-Owner: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Aaron Durbin
adurbin@chromium.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Tue, 26 May 2020 09:47:02 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment