Browse Source

build(fconf)!: clean up source collection

Including the FCONF Makefile today automatically places the FCONF
sources into the source list of the BL1 and BL2 images. This may be
undesirable if, for instance, FCONF is only required for BL31.

This change moves the BL1 and BL2 source appends out of the common
Makefile to where they are required.

BREAKING CHANGE: FCONF is no longer added to BL1 and BL2 automatically
when the FCONF Makefile (`fconf.mk`) is included. When including this
Makefile, consider whether you need to add `${FCONF_SOURCES}` and
`${FCONF_DYN_SOURCES}` to `BL1_SOURCES` and `BL2_SOURCES`.

Change-Id: Ic028eabb7437ae95a57c5bcb7821044d31755c77
Signed-off-by: Chris Kay <chris.kay@arm.com>
pull/1981/head
Chris Kay 4 years ago
parent
commit
e04da4c8e1
  1. 11
      lib/fconf/fconf.mk
  2. 3
      plat/arm/board/a5ds/platform.mk
  3. 3
      plat/arm/board/fvp_ve/platform.mk
  4. 3
      plat/arm/common/arm_common.mk

11
lib/fconf/fconf.mk

@ -1,12 +1,13 @@
# #
# Copyright (c) 2019-2020, ARM Limited. All rights reserved. # Copyright (c) 2019-2021, ARM Limited. All rights reserved.
# #
# SPDX-License-Identifier: BSD-3-Clause # SPDX-License-Identifier: BSD-3-Clause
# #
# Add Firmware Configuration files include common/fdt_wrappers.mk
FCONF_SOURCES := lib/fconf/fconf.c FCONF_SOURCES := lib/fconf/fconf.c
FCONF_DYN_SOURCES := lib/fconf/fconf_dyn_cfg_getter.c FCONF_SOURCES += ${FDT_WRAPPERS_SOURCES}
BL1_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES} FCONF_DYN_SOURCES := lib/fconf/fconf_dyn_cfg_getter.c
BL2_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES} FCONF_DYN_SOURCES += ${FDT_WRAPPERS_SOURCES}

3
plat/arm/board/a5ds/platform.mk

@ -8,6 +8,9 @@
include common/fdt_wrappers.mk include common/fdt_wrappers.mk
include lib/fconf/fconf.mk include lib/fconf/fconf.mk
BL1_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
BL2_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
# Add `libfdt` and Arm common helpers required for Dynamic Config # Add `libfdt` and Arm common helpers required for Dynamic Config
include lib/libfdt/libfdt.mk include lib/libfdt/libfdt.mk

3
plat/arm/board/fvp_ve/platform.mk

@ -127,6 +127,9 @@ endif
# Firmware Configuration Framework sources # Firmware Configuration Framework sources
include lib/fconf/fconf.mk include lib/fconf/fconf.mk
BL1_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
BL2_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
# Add `libfdt` and Arm common helpers required for Dynamic Config # Add `libfdt` and Arm common helpers required for Dynamic Config
include lib/libfdt/libfdt.mk include lib/libfdt/libfdt.mk

3
plat/arm/common/arm_common.mk

@ -258,6 +258,9 @@ BL2_SOURCES += drivers/delay_timer/delay_timer.c \
# Firmware Configuration Framework sources # Firmware Configuration Framework sources
include lib/fconf/fconf.mk include lib/fconf/fconf.mk
BL1_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
BL2_SOURCES += ${FCONF_SOURCES} ${FCONF_DYN_SOURCES}
# Add `libfdt` and Arm common helpers required for Dynamic Config # Add `libfdt` and Arm common helpers required for Dynamic Config
include lib/libfdt/libfdt.mk include lib/libfdt/libfdt.mk

Loading…
Cancel
Save