br101 / zephyr-dw3000-decadriver

Zephyr Driver for Qorvo/Decawave DW3000
42 stars 19 forks source link

Support for Cortex M33 #8

Closed KyleKotowick closed 7 months ago

KyleKotowick commented 8 months ago

I'm working with this using the nRF5340, which has an M33 processor. It seems that the library selection is hardcoded for Cortex M4 processors.

Is there a way to determine which type of processor the board uses and select the library based on that?

br101 commented 8 months ago

That's because the binary only part of the library is only released for this processor. Talk to Qorvo...

KyleKotowick commented 8 months ago

I'm not quite following. In the lib directory it has M33 binaries as well. Which components are missing?

Frank-Buss commented 8 months ago

My company has it working for nRF5340, with Zephyr and their latest SDK. I needed the following libraries:

zephyr_library_import(dwtlib ${drivers_dir}/uwb/Qorvo/Libs/dwt_uwb_driver/lib/libdwt_uwb_driver-m4-hfp-6.0.14.a)
zephyr_library_import(niqlib ${drivers_dir}/uwb/Qorvo/Libs/niq/libniq-m4-hfp-1.1.0.0.a)
zephyr_library_import(maclib ${drivers_dir}/uwb/Qorvo/Libs/uwbstack/uwbstack_lib/lib/cmsis_v1/embedded_mac/libembedded_mac_arm-m4-hfp_R11.9.2.a)

and also the settings CONFIG_FPU=y and CONFIG_FP_HARDABI=y. Maybe works for this project as well. Or you could also port their SDK, needs a few weeks.

br101 commented 7 months ago

I'm not quite following. In the lib directory it has M33 binaries as well. Which components are missing?

Oh, okay. Sorry, I don't actively use this project since quite a while... Then, it should be possible to adapt to use these libraries in that case...

vdjajaj commented 7 months ago

I have added the pull request that allows for the usage of the currently newest nRF Connect SDK (version 2.6.0) and the ability to use the nRF5340 SoC. However, it wasn't extensively tested (it successfully compiles, and I have tested if it reads the DW3000 module's DEV_ID and if dwt_check_dev_id API function returns DWT_SUCCESS and it does).

br101 commented 7 months ago

Thank you, it's merged!