mirror of
https://github.com/RIOT-OS/RIOT.git
synced 2024-12-29 04:50:03 +01:00
e0a5860bb7
The `periph_flash_common` feature was only defined here to trigger inclusion of a source file with common functions. It even only defines private symbols `_lock` and `_unlock` so no reason to expose it to the build system. And in practice, all stm cpus providing `periph_flashpage` or `periph_eeprom` were required to provide `periph_flash_common` to allow including it. The previous implementation was only parsing in the modules were in `FEATURES_REQUIRED` wich did not take cases of `FEATURES_OPTIONAL` into account. And also, in the same time, as the dependencies was declared in `Makefile.include` it was processed before `Makefile.dep` so never handled cases where a module could depend on `periph_flashpage` or `periph_eeprom` feature. It is replaced by selecting the common source file when module using it are included. The now useless feature `periph_flash_common` is removed from `FEATURES_PROVIDED`.
19 lines
506 B
Makefile
19 lines
506 B
Makefile
MODULE = stm32_common_periph
|
|
|
|
# Select the specific implementation for `periph_i2c`
|
|
ifneq (,$(filter periph_i2c,$(USEMODULE)))
|
|
ifneq (,$(filter $(CPU),stm32f0 stm32f3 stm32f7 stm32l0 stm32l4))
|
|
SRC += i2c_1.c
|
|
else # stm32f1/f2/f4/l1
|
|
SRC += i2c_2.c
|
|
endif
|
|
endif
|
|
|
|
# flashpage and eeprom periph implementations share flash lock/unlock functions
|
|
# defined in flash_common.c
|
|
ifneq (,$(filter periph_flashpage periph_eeprom,$(USEMODULE)))
|
|
SRC += flash_common.c
|
|
endif
|
|
|
|
include $(RIOTMAKE)/periph.mk
|