<p>Nico Huber has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/26871">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">drivers/spi: Treat all stages commonly when adding drivers<br><br>Clean it up and fix linking of flashconsole in postcar stage with<br>`CONFIG_SPI_FLASH_INCLUDE_ALL_DRIVERS=y`.<br><br>Change-Id: Ib8bdb824bfcf2d31ac696e39f797c4355b765756<br>Signed-off-by: Nico Huber <nico.h@gmx.de><br>---<br>M src/drivers/spi/Makefile.inc<br>1 file changed, 28 insertions(+), 99 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/26871/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/drivers/spi/Makefile.inc b/src/drivers/spi/Makefile.inc</span><br><span>index cd404f4..e55233e 100644</span><br><span>--- a/src/drivers/spi/Makefile.inc</span><br><span>+++ b/src/drivers/spi/Makefile.inc</span><br><span>@@ -7,105 +7,34 @@</span><br><span> smm-y += spiconsole.c</span><br><span> endif</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-ifeq ($(CONFIG_CONSOLE_SPI_FLASH),y)</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-y += flashconsole.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-y += flashconsole.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-y += flashconsole.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-y += flashconsole.c</span><br><span style="color: hsl(0, 100%, 40%);">-postcar-y += flashconsole.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-y += flashconsole.c</span><br><span style="color: hsl(0, 100%, 40%);">-endif</span><br><span style="color: hsl(120, 100%, 40%);">+# add SPI drivers per stage</span><br><span style="color: hsl(120, 100%, 40%);">+# $1 stage name</span><br><span style="color: hsl(120, 100%, 40%);">+# $2 optional _EARLY suffix for CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP(_EARLY)</span><br><span style="color: hsl(120, 100%, 40%);">+define add_spi_stage</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-y += spi-generic.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-y += bitbang.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_COMMON_CBFS_SPI_WRAPPER) += cbfs_spi.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP$(2)) += boot_device_rw_nommap.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_CONSOLE_SPI_FLASH) += flashconsole.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_ADESTO) += adesto.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_AMIC) += amic.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_ATMEL) += atmel.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_EON) += eon.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_SPANSION) += spansion.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_SST) += sst.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_STMICRO) += stmicro.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FLASH_WINBOND) += winbond.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(1)-$(CONFIG_SPI_FRAM_RAMTRON) += ramtron.c</span><br><span style="color: hsl(120, 100%, 40%);">+endef</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-y += spi-generic.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-y += bitbang.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_COMMON_CBFS_SPI_WRAPPER) += cbfs_spi.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP_EARLY) += boot_device_rw_nommap.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_ADESTO) += adesto.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_AMIC) += amic.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_ATMEL) += atmel.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_EON) += eon.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_SPANSION) += spansion.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_SST) += sst.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_STMICRO) += stmicro.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FLASH_WINBOND) += winbond.c</span><br><span style="color: hsl(0, 100%, 40%);">-bootblock-$(CONFIG_SPI_FRAM_RAMTRON) += ramtron.c</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-y += spi-generic.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-y += bitbang.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_COMMON_CBFS_SPI_WRAPPER) += cbfs_spi.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP_EARLY) += boot_device_rw_nommap.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_ADESTO) += adesto.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_AMIC) += amic.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_ATMEL) += atmel.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_EON) += eon.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_SPANSION) += spansion.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_SST) += sst.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_STMICRO) += stmicro.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FLASH_WINBOND) += winbond.c</span><br><span style="color: hsl(0, 100%, 40%);">-romstage-$(CONFIG_SPI_FRAM_RAMTRON) += ramtron.c</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-y += spi-generic.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-y += bitbang.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_COMMON_CBFS_SPI_WRAPPER) += cbfs_spi.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP_EARLY) += boot_device_rw_nommap.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_ADESTO) += adesto.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_AMIC) += amic.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_ATMEL) += atmel.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_EON) += eon.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_SPANSION) += spansion.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_SST) += sst.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_STMICRO) += stmicro.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FLASH_WINBOND) += winbond.c</span><br><span style="color: hsl(0, 100%, 40%);">-verstage-$(CONFIG_SPI_FRAM_RAMTRON) += ramtron.c</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-y += spi-generic.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-y += bitbang.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_COMMON_CBFS_SPI_WRAPPER) += cbfs_spi.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP) += boot_device_rw_nommap.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_ADESTO) += adesto.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_AMIC) += amic.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_ATMEL) += atmel.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_EON) += eon.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_SPANSION) += spansion.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_SST) += sst.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_STMICRO) += stmicro.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FLASH_WINBOND) += winbond.c</span><br><span style="color: hsl(0, 100%, 40%);">-ramstage-$(CONFIG_SPI_FRAM_RAMTRON) += ramtron.c</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(120, 100%, 40%);">+$(eval $(call add_spi_stage,bootblock,_EARLY))</span><br><span style="color: hsl(120, 100%, 40%);">+$(eval $(call add_spi_stage,romstage,_EARLY))</span><br><span style="color: hsl(120, 100%, 40%);">+$(eval $(call add_spi_stage,verstage,_EARLY))</span><br><span style="color: hsl(120, 100%, 40%);">+$(eval $(call add_spi_stage,postcar,_EARLY))</span><br><span style="color: hsl(120, 100%, 40%);">+$(eval $(call add_spi_stage,ramstage))</span><br><span> ifeq ($(CONFIG_SPI_FLASH_SMM),y)</span><br><span style="color: hsl(0, 100%, 40%);">-smm-y += spi-generic.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-y += bitbang.c</span><br><span style="color: hsl(0, 100%, 40%);">-# SPI flash driver interface</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP) += boot_device_rw_nommap.c</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-# drivers</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_ADESTO) += adesto.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_AMIC) += amic.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_ATMEL) += atmel.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_EON) += eon.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_SPANSION) += spansion.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_SST) += sst.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_STMICRO) += stmicro.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FLASH_WINBOND) += winbond.c</span><br><span style="color: hsl(0, 100%, 40%);">-smm-$(CONFIG_SPI_FRAM_RAMTRON) += ramtron.c</span><br><span style="color: hsl(120, 100%, 40%);">+$(eval $(call add_spi_stage,smm))</span><br><span> endif</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-postcar-y += spi-generic.c</span><br><span style="color: hsl(0, 100%, 40%);">-postcar-y += bitbang.c</span><br><span style="color: hsl(0, 100%, 40%);">-postcar-$(CONFIG_BOOT_DEVICE_SPI_FLASH_RW_NOMMAP_EARLY) += boot_device_rw_nommap.c</span><br><span style="color: hsl(0, 100%, 40%);">-postcar-$(CONFIG_SPI_FLASH) += spi_flash.c</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/26871">change 26871</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/26871"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ib8bdb824bfcf2d31ac696e39f797c4355b765756 </div>
<div style="display:none"> Gerrit-Change-Number: 26871 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Nico Huber <nico.h@gmx.de> </div>