arduino-libraries / Arduino_OV767X

Official OV767X Library for Arduino , currently supports OV7670 and OV7675 cameras
https://www.arduino.cc/reference/en/libraries/arduino_ov767x/
GNU General Public License v2.0
68 stars 25 forks source link

undefined reference `digitalPinToPinName` with mbed@2.5.2 #20

Closed rajames closed 2 years ago

rajames commented 2 years ago

Using mbed_nano@2.5.2 I get the following build errors:

/tmp/arduino-sketch-DA796E73E66DC4EC175B4978B42EC20B/libraries/Arduino_OV767X/objs.a(OV767X.cpp.o): In function `OV767X::begin(int, int, int)':
/home/raul/Arduino/libraries/Arduino_OV767X/src/OV767X.cpp:138: undefined reference to `digitalPinToPinName(unsigned char)'                                                          
/home/raul/Arduino/libraries/Arduino_OV767X/src/OV767X.cpp:139: undefined reference to `digitalPinToPinName(unsigned char)'
/home/raul/Arduino/libraries/Arduino_OV767X/src/OV767X.cpp:140: undefined reference to `digitalPinToPinName(unsigned char)'
/home/raul/Arduino/libraries/Arduino_OV767X/src/OV767X.cpp:141: undefined reference to `digitalPinToPinName(unsigned char)'
/home/raul/Arduino/libraries/Arduino_OV767X/src/OV767X.cpp:142: undefined reference to `digitalPinToPinName(unsigned char)'
/tmp/arduino-sketch-DA796E73E66DC4EC175B4978B42EC20B/libraries/Arduino_OV767X/objs.a(OV767X.cpp.o):/home/raul/Arduino/libraries/Arduino_OV767X/src/OV767X.cpp:143: more undefined references to `digitalPinToPinName(unsigned char)' follow
collect2: error: ld returned 1 exit status
github-actions[bot] commented 2 years ago

Memory usage change @ 0acb788023f584e09353ef4d55d65e3af33dc880

Board flash % RAM for global variables %
arduino:mbed_nano:nano33ble 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table Board|examples/CameraCapture
flash|%|examples/CameraCapture
RAM for global variables|%|examples/CameraCaptureRawBytes
flash|%|examples/CameraCaptureRawBytes
RAM for global variables|%|examples/CameraTestPattern
flash|%|examples/CameraTestPattern
RAM for global variables|%|examples/ConnectionTest
flash|%|examples/ConnectionTest
RAM for global variables|% -|-|-|-|-|-|-|-|-|-|-|-|-|-|-|-|- arduino:mbed_nano:nano33ble|0|0.0|0|0.0|0|0.0|0|0.0|0|0.0|0|0.0|0|0.0|0|0.0
Click for full report CSV ``` Board,examples/CameraCapture
flash,%,examples/CameraCapture
RAM for global variables,%,examples/CameraCaptureRawBytes
flash,%,examples/CameraCaptureRawBytes
RAM for global variables,%,examples/CameraTestPattern
flash,%,examples/CameraTestPattern
RAM for global variables,%,examples/ConnectionTest
flash,%,examples/ConnectionTest
RAM for global variables,% arduino:mbed_nano:nano33ble,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0 ```
rajames commented 2 years ago

any idea when this can be merged?

facchinm commented 2 years ago

@rajames that include shouldn't be used by libraries since it's not part of the APIs and in fact the error you get is arising from some kind of non standard compilation process. If you compile with normal arduino IDE / CLI and core 2.5.2 it will finish cleanly. I can help sorting out the "original" issue though :wink: