Patch Set 3:

Should such changes of defaults in chipset code be announced to the list, or be visibly documented in the release notes?

I don't think announcing in the list is a good idea. This is easily missed.

Listing this type of information in the general readme for the version would result in a huge amount of information that isn't that useful as I think the readme should document the important high level stuff only and not every chipset or board detail.

Perhaps it is a good idea to have a READMEon the component (in this case SoC) level that documents the state and features of the specific code. This can be useful when maintained correctly.

View Change

To view, visit change 37704. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8a06b9fba5ad561d8595292a73136091ab532faa
Gerrit-Change-Number: 37704
Gerrit-PatchSet: 3
Gerrit-Owner: Wim Vervoorn <wvervoorn@eltan.com>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Wim Vervoorn <wvervoorn@eltan.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Thu, 19 Dec 2019 10:33:02 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment