[coreboot-gerrit] Change in coreboot[master]: vendorcode/eltan/security: Use config VENDORCODE_ELTAN_XXX

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33442 )

Change subject: vendorcode/eltan/security: Use config VENDORCODE_ELTAN_XXX
......................................................................

Patch Set 3:

(3 comments)

What's the complete picture here? I see Kconfig variables that are not defined and directories that do not exist?

https://review.coreboot.org/c/coreboot/+/33442/3/src/vendorcode/eltan/securi... 
File src/vendorcode/eltan/security/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/33442/3/src/vendorcode/eltan/securi... 
PS3, Line 17: subdirs-y += mboot
No such directory? (last two)

https://review.coreboot.org/c/coreboot/+/33442/3/src/vendorcode/eltan/securi... 
PS3, Line 19: ifneq ($(filter y,$(CONFIG_VENDORCODE_ELTAN_VBOOT) $(CONFIG_VENDORCODE_ELTAN_MBOOT)),)
No such Kconfig defined?

https://review.coreboot.org/c/coreboot/+/33442/3/src/vendorcode/eltan/securi... 
PS3, Line 26: CPPFLAGS_common += -I$(src)/security/include
No such directory?