[riot-notifications] [RIOT-OS/RIOT] drivers/periph_spi: let spi_acquire return void (#15902)

Hugues Larrive notifications at github.com
Sat Sep 4 02:45:16 CEST 2021


@hugueslarrive commented on this pull request.



> @@ -132,6 +131,7 @@ static uint8_t _trx(cc2538_ssi_t *dev, uint8_t in)
 void spi_transfer_bytes(spi_t bus, spi_cs_t cs, bool cont,
                         const void *out, void *in, size_t len)
 {
+    assert((unsigned)bus < SPI_NUMOF);

Why assert in spi_transfer_bytes instead of spi_acquire for this one ?

-- 
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/15902#pullrequestreview-746524204
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210903/4f4107e5/attachment.htm>


More information about the notifications mailing list