Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

boards: qorvo: add DWM3001CDK support #79019

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mtfurlan
Copy link

@mtfurlan mtfurlan commented Sep 25, 2024

This doensn't add any drivers for the DWM3001C, but it works with
https://github.com/br101/zephyr-dw3000-decadriver/

It's basically a copy of the decawave_dwm1001_dev with the modifications to work with the dwm3001cdk

Copy link

Hello @mtfurlan, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@mtfurlan mtfurlan force-pushed the boards/add-dwm3001cdk branch 2 times, most recently from 20ee81e to 6420f65 Compare September 25, 2024 16:55
Copy link
Author

@mtfurlan mtfurlan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works with the usb cdc console example, but I kinda just modified stuff till it looked right, especially stuff like decawave_dwm3001cdk.yaml

boards/qorvo/decawave_dwm3001cdk/doc/index.rst Outdated Show resolved Hide resolved
@mtfurlan mtfurlan force-pushed the boards/add-dwm3001cdk branch 6 times, most recently from 7ca0de5 to 171ff3d Compare September 25, 2024 23:55
config I2C
default SENSOR

if BOARD_SERIAL_BACKEND_CDC_ACM
Copy link
Author

@mtfurlan mtfurlan Sep 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took this from nrf52840dongle cause it made the tests/subsys/llext/simple test build, I don't really know if it's correct

@mtfurlan mtfurlan force-pushed the boards/add-dwm3001cdk branch 2 times, most recently from 1b9690d to 1fe6210 Compare September 27, 2024 14:30
@zephyrbot zephyrbot added the area: ADC Analog-to-Digital Converter (ADC) label Sep 27, 2024
@mtfurlan
Copy link
Author

So I can either get tests that expect console to build by setting up usb stuff, or I can get tests that test usb stuff to build by not setting up usb stuff

Not really sure what do do about this, some existing boards just don't pass some tests

for example:
west build -p -b nrf52840dongle tests/drivers/udc -T drivers.usb.udc fails because nrf52840dongle uses usb cdc as a console, so gen_isr_tables.py: error: multiple registrations at table_index 39 for irq 39 (0x27)

@mtfurlan mtfurlan force-pushed the boards/add-dwm3001cdk branch 2 times, most recently from fdadb2d to c7650c0 Compare September 27, 2024 16:48
This doensn't add any drivers for the DWM3001C, but it works with
https://github.com/br101/zephyr-dw3000-decadriver/

Signed-off-by: Mara Furland <mara@fur.land>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants