[riot-notifications] [RIOT-OS/RIOT] drivers/pn532: use pseudomodules to select i2c or spi (#15705)

Leandro Lanzieri notifications at github.com
Mon Jan 4 17:02:45 CET 2021


### Contribution description
This PR adapts the way to select SPI and/or I2C support in the pn532 driver, now it uses pseudomodules. That allows to include the peripheral dependencies in the dependency resolution step, and is in line with the common practice. Also, this includes a fix on the signature of a private function (`static void reverse(...)`), which caused a compile error when enabling SPI support on the driver.

### Testing procedure
- The binary should not change
- SPI support should compile successfully
- The required features should be pulled-in automatically by selecting the correspondent module (`pn532_i2c`, `pn532_spi`)

### Issues/PRs references
None

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

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

-- Commit Summary --

  * drivers/pn532: use pseudomodules to select i2c or spi

-- File Changes --

    M drivers/Makefile.dep (10)
    M drivers/include/pn532.h (17)
    M drivers/pn532/pn532.c (15)
    M makefiles/pseudomodules.inc.mk (4)
    M tests/driver_pn532/Makefile (17)
    M tests/driver_pn532/main.c (9)
    M tests/driver_pn532/pn532_params.h (9)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/15705.patch
https://github.com/RIOT-OS/RIOT/pull/15705.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/15705
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210104/ecffbab8/attachment.htm>


More information about the notifications mailing list