[riot-notifications] [RIOT-OS/RIOT] cpu/native: Allow Access to Hardware SPI Bus on Linux (#11352)
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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications