Change in coreboot[master]: tpm: Refactor TPM Kconfig dimensions

[Jes Klinke (Code Review)]

Attention is currently required from: Stefan Ott, Felix Singer, Patrick Rudolph, Jonathan Zhang, Nick Vaccaro, Arthur Heymans, Andrey Petrov, Piotr Król, Nico Huber, Sean Rhodes, Michał Żygowski, Johnny Lin, Christian Walter, Werner Zeh, Alexander Couzens, Yu-Ping Wu, Tim Chu, Frans Hendriks, Tristan Corrick, Jeremy Soller, Angel Pons, Julius Werner, Tim Crawford, Maxim Polyakov, Tim Wawrzynczak.
Jes Klinke has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63424 )

Change subject: tpm: Refactor TPM Kconfig dimensions
......................................................................

Patch Set 5:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/63424/comment/21db02be_c52222f9 
PS3, Line 41:
> Signed-off-by

Done

File src/drivers/i2c/tpm/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/63424/comment/d72ade67_17caef83 
PS3, Line 9: ifeq ($(CONFIG_TPM_ATMEL),y)
           : ramstage-y += tis_atmel.c
           : romstage-y += tis_atmel.c
           : verstage-y += tis_atmel.c
           : bootblock-y += tis_atmel.c
           : postcar-y += tis_atmel.c
           : elif ($(CONFIG_TPM_GOOGLE),y)
           : ramstage-y += cr50.c
           : romstage-y += cr50.c
           : verstage-y += cr50.c
           : bootblock-y += cr50.c
           : postcar-y += cr50.c
           : else
           : ramstage-y += tpm.c
           : romstage-y += tpm.c
           : verstage-y += tpm.c
           : bootblock-y += tpm.c
           : postcar-y += tpm.c
           : endif
> You could use the 'all-y' target here as you add the driver to every stage anyway. […]

Done, thanks for pointing this out.

File src/drivers/pc80/tpm/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/63424/comment/4f713381_05f8bd18 
PS3, Line 1: bootblock-$(CONFIG_LPC_TPM) += tis.c
           : verstage-$(CONFIG_LPC_TPM) += tis.c
           : romstage-$(CONFIG_LPC_TPM) += tis.c
           : ramstage-$(CONFIG_LPC_TPM) += tis.c
           : postcar-$(CONFIG_LPC_TPM) += tis.c
> all-$(CONFIG_LPC_TPM) += tis.c […]

Done

File src/soc/intel/common/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/63424/comment/00a5882b_5f58b5bd 
PS3, Line 23: bootblock-$(CONFIG_TPM_GOOGLE) += tpm_tis.c
            : verstage-$(CONFIG_TPM_GOOGLE) += tpm_tis.c
            : romstage-$(CONFIG_TPM_GOOGLE) += tpm_tis.c
            : ramstage-$(CONFIG_TPM_GOOGLE) += tpm_tis.c
            : postcar-$(CONFIG_TPM_GOOGLE) += tpm_tis.c
> all-$(CONFIG_TPM_GOOGLE)

Done