[riot-notifications] [RIOT-OS/RIOT] cpu/native: Allow Access to Hardware SPI Bus on Linux (#11352)

Frank Hessel notifications at github.com
Tue Apr 9 14:34:16 CEST 2019


@kaspar030 Thank you for the feedback! If I can anyhow support testing, just let me know. I addressed the styling issue already.

Regarding the failing CI build, I assume that the failing tests are now run for the first time for the native CPU (all of them are for drivers which require `PERIPH_SPI` as feature, which is now available). As far as I could see, the reason for the failure is mostly using `char` as type for byte constants or assigning macros which expand to 1-byte expressions to chars, with chars being signed on _native_. This leads to the `overflow in implicit constant conversion` seen in the CI log.

The only other issue that I could see was a binary constant in `tests_driver_adt7310` which creates an error with this toolchain as well.

As that's not directly related to the suggested change, maybe you could suggest what's the best way to address this.

-- 
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/11352#issuecomment-481232161
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190409/8ed43ff9/attachment.html>


More information about the notifications mailing list