-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: move remaining driver related applications to tests/drivers #19568
Conversation
bors merge |
19484: makefiles/arch/msp430.inc.mk: Fix compilation issues with GCC 12 r=maribu a=maribu ### Contribution description This fixes the following compilation issues: /home/maribu/Repos/software/RIOT/cpu/msp430fxyz/periph/gpio.c: In function 'gpio_periph_mode': /home/maribu/Repos/software/RIOT/cpu/msp430fxyz/periph/gpio.c:95:15: error: array subscript 0 is outside array bounds of 'msp_port_isr_t[0]' [-Werror=array-bounds] 95 | sel = &(isrport->SEL); | ^~~~~~~~~~~~~~~ cc1: all warnings being treated as errors by adding `CFLAGS += --param-min-pagesize=0` for GCC 12 (same issue as already fixed for AVR). and: /usr/lib/gcc/msp430-elf/12.2.0/../../../../msp430-elf/bin/ld: warning: /home/maribu/Repos/software/RIOT/cpu/msp430_common/ldscripts/xfa.ld contains output sections; did you forget -T? by adding the missing `-T`. ### Testing procedure The following should still work: - `make BOARD=msb-430 -C examples/hello-world` - `make BOARD=msb-430 -C tests/xfa flash test` ### Issues/PRs references None 19568: tests: move remaining driver related applications to tests/drivers r=aabadie a=aabadie 19575: treewide: fix format specifiers r=maribu a=maribu ### Contribution description This brings format specifiers and the passed type back into sync. This won't change observable behavior in one case, and won't even change machine code in the other. But formally, this fixes bugs. Co-authored-by: Marian Buschsieweke <marian.buschsieweke@ovgu.de> Co-authored-by: Alexandre Abadie <alexandre.abadie@inria.fr>
Build failed (retrying...): |
bors merge |
Already running a review |
19484: makefiles/arch/msp430.inc.mk: Fix compilation issues with GCC 12 r=maribu a=maribu ### Contribution description This fixes the following compilation issues: /home/maribu/Repos/software/RIOT/cpu/msp430fxyz/periph/gpio.c: In function 'gpio_periph_mode': /home/maribu/Repos/software/RIOT/cpu/msp430fxyz/periph/gpio.c:95:15: error: array subscript 0 is outside array bounds of 'msp_port_isr_t[0]' [-Werror=array-bounds] 95 | sel = &(isrport->SEL); | ^~~~~~~~~~~~~~~ cc1: all warnings being treated as errors by adding `CFLAGS += --param-min-pagesize=0` for GCC 12 (same issue as already fixed for AVR). and: /usr/lib/gcc/msp430-elf/12.2.0/../../../../msp430-elf/bin/ld: warning: /home/maribu/Repos/software/RIOT/cpu/msp430_common/ldscripts/xfa.ld contains output sections; did you forget -T? by adding the missing `-T`. ### Testing procedure The following should still work: - `make BOARD=msb-430 -C examples/hello-world` - `make BOARD=msb-430 -C tests/xfa flash test` ### Issues/PRs references None 19565: tests: move core related applications to their own tests/core/ folder r=aabadie a=aabadie 19568: tests: move remaining driver related applications to tests/drivers r=aabadie a=aabadie 19575: treewide: fix format specifiers r=maribu a=maribu ### Contribution description This brings format specifiers and the passed type back into sync. This won't change observable behavior in one case, and won't even change machine code in the other. But formally, this fixes bugs. Co-authored-by: Marian Buschsieweke <marian.buschsieweke@ovgu.de> Co-authored-by: Alexandre Abadie <alexandre.abadie@inria.fr>
Build failed (retrying...): |
19568: tests: move remaining driver related applications to tests/drivers r=aabadie a=aabadie 19575: treewide: fix format specifiers r=maribu a=maribu ### Contribution description This brings format specifiers and the passed type back into sync. This won't change observable behavior in one case, and won't even change machine code in the other. But formally, this fixes bugs. Co-authored-by: Alexandre Abadie <alexandre.abadie@inria.fr> Co-authored-by: Marian Buschsieweke <marian.buschsieweke@ovgu.de>
Build failed (retrying...): |
19568: tests: move remaining driver related applications to tests/drivers r=aabadie a=aabadie 19574: cpu/stm32/periph_gpio: reset PU/PD for ADC channels r=maribu a=gschorcht ### Contribution description This PR provides a small fix that is relevant when a GPIO has been used as input/output with a pull resistor before it is initialized as an ADC channel. The PU/PD configuration has to be `0b00` for analog outputs which is corresponds to the reset state. However, if the GPIO is not in the reset state but was used digital input/output with any pull resistor, the PU/PD configuration has also to be reset to use it as ADC channel. ### Testing procedure - Green CI - The `periph_adc` test application should still work for any board that supports the `periph_adc` feature. ### Issues/PRs references Co-authored-by: Alexandre Abadie <alexandre.abadie@inria.fr> Co-authored-by: Gunar Schorcht <gunar@schorcht.net>
Build failed (retrying...): |
Build failed: |
bors merge |
Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Contribution description
While working on other similar PRs, I noticed that some test application related to driver modules (mtd*, disp_dev, touch_dev, saul*) were still located in
tests/
although they could be moved totests/drivers
as well.Testing procedure
Green CI
Issues/PRs references
follow-up of #19435