Skip to content

Commit

Permalink
Merge #19023
Browse files Browse the repository at this point in the history
19023: pkg/tinyusb: fix Kconfig problems r=maribu a=gschorcht

### Contribution description

This PR tries to fix the [problems with `tinyUSB` in nightlies](https://ci.riot-os.org/details/d11b383ba0cd45048ed6f8d98abbc0de).
- The `tinyusb_device` feature had to be moved from `boards/common/arduino-zero` definition to the `boards/arduino-zero` definition because the common `arduino-zero` features are also used by `wemos-zero` which uses the `highlevel_stdio` feature via the `stdio_cdc_acm` module.
- The `tinyusb_device` feature had to be removed from Kconfig of board `stm32f429i-disco` since it uses the `highlevel_stdio` feature via the `stdio_cdc_acm` module.

### Testing procedure

Green CI

### Issues/PRs references


Co-authored-by: Gunar Schorcht <gunar@schorcht.net>
  • Loading branch information
bors[bot] and gschorcht committed Dec 13, 2022
2 parents dc97657 + b805709 commit a0f823c
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 4 deletions.
1 change: 1 addition & 0 deletions boards/arduino-zero/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ config BOARD
config BOARD_ARDUINO_ZERO
bool
default y
select HAS_TINYUSB_DEVICE
select BOARD_COMMON_ARDUINO_ZERO

source "$(RIOTBOARD)/common/arduino-zero/Kconfig"
2 changes: 2 additions & 0 deletions boards/arduino-zero/Makefile.features
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
include $(RIOTBOARD)/common/arduino-zero/Makefile.features

FEATURES_PROVIDED += tinyusb_device
1 change: 0 additions & 1 deletion boards/common/arduino-zero/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ config BOARD_COMMON_ARDUINO_ZERO
select HAS_PERIPH_USBDEV
select HAS_ARDUINO
select HAS_ARDUINO_PWM
select HAS_TINYUSB_DEVICE

select HAVE_SAUL_GPIO

Expand Down
1 change: 0 additions & 1 deletion boards/common/arduino-zero/Makefile.features
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,3 @@ FEATURES_PROVIDED += periph_usbdev
# Various other features (if any)
FEATURES_PROVIDED += arduino
FEATURES_PROVIDED += arduino_pwm
FEATURES_PROVIDED += tinyusb_device
2 changes: 1 addition & 1 deletion boards/stm32f429i-disc1/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ config BOARD_STM32F429I_DISC1

# Put other features for this board (in alphabetical order)
select HAS_RIOTBOOT
select HAS_TINYUSB_DEVICE
select HAS_TINYUSB_DEVICE if !BOARD_STM32F429I_DISCO

# Clock configuration
select BOARD_HAS_HSE
Expand Down
5 changes: 4 additions & 1 deletion boards/stm32f429i-disc1/Makefile.features
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,7 @@ FEATURES_PROVIDED += periph_usbdev

# Put other features for this board (in alphabetical order)
FEATURES_PROVIDED += riotboot
FEATURES_PROVIDED += tinyusb_device

ifneq (,$(filter stm32f429i-disc1,$(BOARD)))
FEATURES_PROVIDED += tinyusb_device
endif

0 comments on commit a0f823c

Please sign in to comment.