Commit c452ba15 authored by Masahiro Yamada's avatar Masahiro Yamada
Browse files

fconf: exclude fconf_dyn_cfg_getter.c from BL1_SOURCES



fconf_dyn_cfg_getter.c calls FCONF_REGISTER_POPULATOR(), which populates
the fconf_populator structure.

However, bl1/bl1.ld.S does not have:

        __FCONF_POPULATOR_START__ = .;
        KEEP(*(.fconf_populator))
        __FCONF_POPULATOR_END__ = .;

So, this is not linked to bl1.elf

We could change either bl1/bl1.lds.S or lib/fconf/fconf.mk to make
them consistent.

I chose to fix up fconf.mk to keep the current behavior.

This is a groundwork to factor out the common code from linker scripts.

Change-Id: I07b7ad4db4ec77b57acf1588fffd0b06306d7293
Signed-off-by: default avatarMasahiro Yamada <yamada.masahiro@socionext.com>
parent 85ee795c
...@@ -5,8 +5,8 @@ ...@@ -5,8 +5,8 @@
# #
# Add Firmware Configuration files # Add Firmware Configuration files
FCONF_SOURCES := lib/fconf/fconf.c \ FCONF_SOURCES := lib/fconf/fconf.c
lib/fconf/fconf_dyn_cfg_getter.c FCONF_DYN_SOURCES := lib/fconf/fconf_dyn_cfg_getter.c
BL1_SOURCES += ${FCONF_SOURCES} BL1_SOURCES += ${FCONF_SOURCES}
BL2_SOURCES += ${FCONF_SOURCES} BL2_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment