search

SolidGeek / VescUart

An Arduino library for interfacing with the VESC over UART
GNU General Public License v3.0
177 stars 92 forks source link

Compile Problem on Arduino BLE Sense (nRF52840) #22

Closed Crelix-M closed 3 years ago

Crelix-M commented 3 years ago

I'm trying to compile even just the initiate Part the Library. It seems to confuse the typedef of NRF_POWER in datatypes.h (line 385) of the VescUart Lib with the Peripheral declaration from the mbed Part of the Arduino Core in nrf52840.h (line 2405). I get the following Error Message:

**C:....platformio\packages\framework-arduino-nrf52-mbedos\cores\arduino/mbed/targets/TARGET_NORDIC/TARGET_NRF5x/TARGET_SDK_15_0/modules/nrfx/mdk/nrf52840.h:2405:67: error: expected ')' before '' token

define NRF_POWER ((NRF_POWER_Type *) NRF_POWER_BASE)***

.pio\libdeps\nano33ble\VescUart\src/datatypes.h:385:3: note: in expansion of macro 'NRF_POWER' } NRF_POWER;

Any Idea how to solve this? I either missed a trivial solution or it exceeds my experience as I don't really know where to start troubleshooting

Crelix-M commented 3 years ago

Alright I just noticed that datatypes.h was changed in the Development Branch, switching to it seems to fix it. Had to add Curly Braces to the Switch Case in VescUart.cpp (COMM_GET_VALUES) to fix a "jump to case label" error though Corresponds with: Pull request https://github.com/SolidGeek/VescUart/pull/20#issue-461807029