oxullo / Arduino-MAX30100

Arduino library for MAX30100, integrated oximeter and heart rate sensor
GNU General Public License v3.0
188 stars 134 forks source link

MAX30100 and BLE Nano #3

Closed AzraelNB closed 7 years ago

AzraelNB commented 7 years ago

I'm trying to get your code to compile on a BLE Nano from RedBearLabs. Unfortunately I keep running into this issue:

MAX30100.cpp:30:10: error: 'class TwoWire' has no member named 'setClock'

 Wire.setClock(I2C_BUS_SPEED);

      ^

exit status 1 Error compiling for board BLE Nano(V1.5 32KB).

I realise that the problem lies with the adapted wire library for the BLE Nano, but I was wondering if you had any thoughts on how I could address this problem?

oxullo commented 7 years ago

@AzraelNB as far as I can see here: https://github.com/RedBearLab/nRF51822-Arduino/blob/S130/arduino-1.6.x/hardware/RBL/RBL_nRF51822/libraries/Wire/Wire.cpp yes, there's no setClock() method. Check if 970e2d404c87b177beb398eaad9346b317eb68e9 works for you. I haven't tested it at all, though!

AzraelNB commented 7 years ago

Thanks! Well, it resolved that issue, but there seem to be a whole number of other issues popping up now too:/ Seems getting it to work on the BLE Nano will take some work.

oxullo commented 7 years ago

I imagined.. In case you see the chance for a PR, I'll be glad to review it.