Attention is currently required from: Furquan Shaikh, Karthik Ramasubramanian, Felix Held.
1 comment:
File src/soc/amd/common/block/spi/fch_spi.c:
if (CONFIG(EM100))
fch_spi_config_em100_modes();
else
fch_spi_config_mb_modes();
This should be completely dropped and instead coreboot should rely on using EFS fields to determine […]
Do you realize that EFS only configures some of the speeds, not all of them?
Do you understand that this would allow us to use EM100 with our pre-built images instead of requiring a rebuild?
Do you realize that not all boards being used right now can run at the higher speeds that we're transitioning to?
With all of that in mind, and understanding that we can't JUST use EFS for configuration, do you still think that this should be dropped?
To view, visit change 56818. To unsubscribe, or for help writing mail filters, visit settings.