Skip to content

cpu/native: migrate periph_rtc to ztimer#17125

Merged
aabadie merged 1 commit intoRIOT-OS:masterfrom
aabadie:pr/cpu/native_rtc_ztimer
Nov 11, 2021
Merged

cpu/native: migrate periph_rtc to ztimer#17125
aabadie merged 1 commit intoRIOT-OS:masterfrom
aabadie:pr/cpu/native_rtc_ztimer

Conversation

@aabadie
Copy link
Copy Markdown
Contributor

@aabadie aabadie commented Nov 3, 2021

Contribution description

This PR migrate the RTC periph of native from xtimer to ztimer.

Testing procedure

  • Green CI
  • tests/periph_rtc still works on native:
Details
$ BUILD_IN_DOCKER=1 make -C tests/periph_rtc all test BOARD=native --no-print-directory 
Launching build container using image "riot/riotbuild:latest".
docker run --rm --tty --user $(id -u) -v '/usr/share/zoneinfo/Europe/Paris:/etc/localtime:ro' -v '/work/riot/RIOT:/data/riotbuild/riotbase:delegated' -e 'RIOTBASE=/data/riotbuild/riotbase' -e 'CCACHE_BASEDIR=/data/riotbuild/riotbase' -e 'BUILD_DIR=/data/riotbuild/riotbase/build' -e 'RIOTPROJECT=/data/riotbuild/riotbase' -e 'RIOTCPU=/data/riotbuild/riotbase/cpu' -e 'RIOTBOARD=/data/riotbuild/riotbase/boards' -e 'RIOTMAKE=/data/riotbuild/riotbase/makefiles'      -e 'BOARD=native'  -w '/data/riotbuild/riotbase/tests/periph_rtc/' 'riot/riotbuild:latest' make 'BOARD=native'    all
Building application "tests_periph_rtc" for "native" with MCU "native".

"make" -C /data/riotbuild/riotbase/boards/native
"make" -C /data/riotbuild/riotbase/boards/native/drivers
"make" -C /data/riotbuild/riotbase/core
"make" -C /data/riotbuild/riotbase/cpu/native
"make" -C /data/riotbuild/riotbase/cpu/native/periph
"make" -C /data/riotbuild/riotbase/cpu/native/stdio_native
"make" -C /data/riotbuild/riotbase/drivers
"make" -C /data/riotbuild/riotbase/drivers/periph_common
"make" -C /data/riotbuild/riotbase/sys
"make" -C /data/riotbuild/riotbase/sys/auto_init
"make" -C /data/riotbuild/riotbase/sys/div
"make" -C /data/riotbuild/riotbase/sys/frac
"make" -C /data/riotbuild/riotbase/sys/test_utils/interactive_sync
"make" -C /data/riotbuild/riotbase/sys/xtimer
"make" -C /data/riotbuild/riotbase/sys/ztimer
   text	   data	    bss	    dec	    hex	filename
  43191	    736	  47976	  91903	  166ff	/data/riotbuild/riotbase/tests/periph_rtc/bin/native/tests_periph_rtc.elf
r
/work/riot/RIOT/tests/periph_rtc/bin/native/tests_periph_rtc.elf /dev/ttyACM0 
RIOT native interrupts/signals initialized.
LED_RED_OFF
LED_GREEN_ON
RIOT native board initialized.
RIOT native hardware initialization complete.

Help: Press s to start test, r to print it is ready
READY
s
START
main(): This is RIOT! (Version: 2022.01-devel-344-g940f7-pr/cpu/native_rtc_ztimer)

RIOT RTC low-level driver test
This test will display 'Alarm!' every 2 seconds for 4 times
Native RTC initialized.
  Setting clock to   2020-02-28 23:59:57
Clock value is now   2020-02-28 23:59:57
  Setting alarm to   2020-02-28 23:59:59
   Alarm is set to   2020-02-28 23:59:59
  Alarm cleared at   2020-02-28 23:59:57
       No alarm at   2020-02-28 23:59:59
  Setting alarm to   2020-02-28 23:59:61

Alarm!
Alarm!
Alarm!
Alarm!

Issues/PRs references

Based on #17165

@github-actions github-actions bot added Area: cpu Area: CPU/MCU ports Platform: native Platform: This PR/issue effects the native platform labels Nov 3, 2021
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 3, 2021
@aabadie aabadie force-pushed the pr/cpu/native_rtc_ztimer branch 3 times, most recently from a57b15c to 75adc65 Compare November 4, 2021 15:21
@aabadie aabadie force-pushed the pr/cpu/native_rtc_ztimer branch from 75adc65 to f9838de Compare November 9, 2021 16:27
@aabadie aabadie added the State: waiting for other PR State: The PR requires another PR to be merged first label Nov 9, 2021
@github-actions github-actions bot added Area: examples Area: Example Applications Area: Kconfig Area: Kconfig integration Area: sys Area: System Area: tests Area: tests and testing framework Area: timers Area: timer subsystems labels Nov 9, 2021
@aabadie aabadie removed the State: waiting for other PR State: The PR requires another PR to be merged first label Nov 11, 2021
@aabadie aabadie force-pushed the pr/cpu/native_rtc_ztimer branch from f9838de to 326b0c9 Compare November 11, 2021 09:55
@github-actions github-actions bot removed Area: Kconfig Area: Kconfig integration Area: tests Area: tests and testing framework Area: sys Area: System Area: timers Area: timer subsystems Area: examples Area: Example Applications labels Nov 11, 2021
@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented Nov 11, 2021

Rebased since this PR no longer waits for #17165

@aabadie aabadie merged commit a85853a into RIOT-OS:master Nov 11, 2021
@aabadie aabadie deleted the pr/cpu/native_rtc_ztimer branch November 11, 2021 16:26
@fjmolinas fjmolinas added this to the Release 2022.01 milestone Nov 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: native Platform: This PR/issue effects the native platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants