Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37704 )
Change subject: soc/intel/skylake: Change SA_PCIEX_LENGTH to 256MB ......................................................................
Patch Set 3:
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.