[riot-notifications] [RIOT-OS/RIOT] drivers, tests: use xtimer_msleep() (#15385)

benpicco notifications at github.com
Wed Nov 4 23:08:02 CET 2020


<!--
The RIOT community cares a lot about code quality.
Therefore, before describing what your contribution is about, we would like
you to make sure that your modifications are compliant with the RIOT
coding conventions, see https://github.com/RIOT-OS/RIOT/wiki/Coding-conventions.
-->

### Contribution description

Use the `xtimer_msleep()` function instead of `xtimer_usleep(… * US_PER_MS)`.
This only does the obvious replacements where both `xtimer_usleep` and `US_PER_MS` appeared in the same line.

### Testing procedure

With `xtimer` nothing will change as `xtimer_msleep()` is just an inline wrapper function that multiplies the sleep time by 1000.
With `ztimer` however now there is the possibility to use the ztimer_msec timer for these sleeps.

Nonetheless, the code should look slightly cleaner.


### Issues/PRs references

<!--
Examples: Fixes #1234. See also #5678. Depends on PR #9876.

Please use keywords (e.g., fixes, resolve) with the links to the issues you
resolved, this way they will be automatically closed when your pull request
is merged. See https://help.github.com/articles/closing-issues-using-keywords/.
-->

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/15385

-- Commit Summary --

  * drivers/ad7746: convert to xtimer_msleep()
  * drivers/aip31068: convert to xtimer_msleep()
  * drivers/ata8520e: convert to xtimer_msleep()
  * drivers/atwinc15x0: convert to xtimer_msleep()
  * drivers/bme680: convert to xtimer_msleep()
  * drivers/dfplayer: convert to xtimer_msleep()
  * drivers/dht: convert to xtimer_msleep()
  * drivers/ds75lx: convert to xtimer_msleep()
  * drivers/dsp0401: convert to xtimer_msleep()
  * drivers/ili9341: convert to xtimer_msleep()
  * drivers/itg320x: convert to xtimer_msleep()
  * drivers/mhz19: convert to xtimer_msleep()
  * drivers/ph_oem: convert to xtimer_msleep()
  * drivers/rn2xx3: convert to xtimer_msleep()
  * drivers/sht2x: convert to xtimer_msleep()
  * drivers/si70xx: convert to xtimer_msleep()
  * drivers/stmpe811: convert to xtimer_msleep()
  * pkg/driver_bme680/contrib: convert to xtimer_msleep()
  * pkg/paho-mqtt: convert to xtimer_msleep()
  * tests/driver_aip31068: convert to xtimer_msleep()
  * tests/driver_bme680: convert to xtimer_msleep()
  * tests/driver_dfplayer: convert to xtimer_msleep()
  * tests/driver_dsp0401: convert to xtimer_msleep()
  * tests/driver_jc42: convert to xtimer_msleep()
  * tests/driver_pca9633: convert to xtimer_msleep()
  * tests/driver_si70xx: convert to xtimer_msleep()
  * tests/event_wait_timeout: convert to xtimer_msleep()
  * tests/evtimer_msg: convert to xtimer_msleep()
  * tests/pkg_umorse: convert to xtimer_msleep()
  * tests/progress_bar: convert to xtimer_msleep()
  * tests/sys_irq_handler: convert to xtimer_msleep()
  * tests/thread_priority_inversion: convert to xtimer_msleep()
  * tests/touch_dev: convert to xtimer_msleep()
  * tests/unittests/tests-fib_sr: convert to xtimer_msleep()

-- File Changes --

    M drivers/ad7746/ad7746.c (2)
    M drivers/aip31068/aip31068.c (4)
    M drivers/ata8520e/ata8520e.c (6)
    M drivers/atwinc15x0/atwinc15x0_bsp.c (2)
    M drivers/bme680/bme680_saul.c (2)
    M drivers/dfplayer/dfplayer_internal.c (2)
    M drivers/dht/dht.c (2)
    M drivers/ds75lx/ds75lx.c (2)
    M drivers/dsp0401/dsp0401.c (4)
    M drivers/ili9341/ili9341.c (6)
    M drivers/itg320x/itg320x.c (4)
    M drivers/mhz19/mhz19_pwm.c (8)
    M drivers/mhz19/mhz19_uart.c (2)
    M drivers/ph_oem/ph_oem.c (2)
    M drivers/rn2xx3/rn2xx3.c (2)
    M drivers/sht2x/sht2x.c (6)
    M drivers/si70xx/si70xx.c (2)
    M drivers/stmpe811/stmpe811.c (4)
    M pkg/driver_bme680/contrib/bme680_hal.c (2)
    M pkg/paho-mqtt/contrib/riot_iface.c (2)
    M tests/driver_aip31068/main.c (28)
    M tests/driver_bme680/main.c (2)
    M tests/driver_dfplayer/main.c (2)
    M tests/driver_dsp0401/main.c (4)
    M tests/driver_jc42/main.c (2)
    M tests/driver_pca9633/main.c (28)
    M tests/driver_si70xx/main.c (2)
    M tests/event_wait_timeout/main.c (8)
    M tests/evtimer_msg/main.c (2)
    M tests/pkg_umorse/main.c (2)
    M tests/progress_bar/main.c (4)
    M tests/sys_irq_handler/main.c (2)
    M tests/thread_priority_inversion/main.c (2)
    M tests/touch_dev/main.c (2)
    M tests/unittests/tests-fib_sr/tests-fib_sr.c (6)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/15385.patch
https://github.com/RIOT-OS/RIOT/pull/15385.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15385
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201104/e93d049c/attachment-0001.htm>


More information about the notifications mailing list