Skip to content

[driver] TI IWR6843 AOP #1336

Merged
salkinium merged 4 commits intomodm-io:developfrom
hshose:feature/iwr6843aop
Mar 7, 2026
Merged

[driver] TI IWR6843 AOP #1336
salkinium merged 4 commits intomodm-io:developfrom
hshose:feature/iwr6843aop

Conversation

@hshose
Copy link
Contributor

@hshose hshose commented Feb 18, 2026

Driver for the TI IWR6843 automotive radar sensor with antenna on package.
The driver is pretty much a 1:1 translation of the rust implementation by @joelsa - therefore naming joel as co-author 🎉

The driver configures the sensor via uart and reads data via a second uart.

I had to add higher uart baudrates to the linux uart driver.

Todos:

<td align="center"><a href="https://modm.io/reference/module/modm-driver-is31fl3733">IS31FL3733</a></td>
<td align="center"><a href="https://modm.io/reference/module/modm-driver-itg3200">ITG3200</a></td>
</tr><tr>
<td align="center"><a href="https://modm.io/reference/module/modm-driver-iwr6843aop">IWR6843AOP</a></td>
Copy link
Contributor

Choose a reason for hiding this comment

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

In theory, this driver should work with all members of the xWRxx4x family that can run the mmWave SDK Demo binary, that is IWR1443, IWR1642, IWR1843, IWR6443 and IWR6843 and their automotive AWR* counterparts.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I cant test with the other chips, but good if this works!

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I just wanted to state this somewhere so that someone looking for a driver for the others can stumble over it.

Copy link
Member

Choose a reason for hiding this comment

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

Elsewhere in modm, drivers are also named after the device family, even if only one device was tested.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

so you want to rename the driver xWRxx4x in the lbuild module? Not sure if this makes it more findable... and the tools/scripts/synchronize_docs.py script pulls the table entry direcetly from the lbuild module name so messing around with the README.md doesn't really work.

I have added a more verbose docstring to the Iwr6843aop class so if someone searches the code, more part numbers show up.

@hshose hshose force-pushed the feature/iwr6843aop branch from f6b0ad7 to e174a1f Compare February 25, 2026 20:36
@hshose
Copy link
Contributor Author

hshose commented Feb 25, 2026

image

After some issues with the 5V rail of the Nucleo dropping and browning out the radar sensor when commanding start measuring, we have a functioning example 🎉

@hshose hshose force-pushed the feature/iwr6843aop branch from 48ac072 to 892da28 Compare February 25, 2026 20:56
@hshose hshose marked this pull request as ready for review March 3, 2026 12:16
@hshose hshose force-pushed the feature/iwr6843aop branch from c2ee791 to 6647786 Compare March 3, 2026 12:23
@hshose
Copy link
Contributor Author

hshose commented Mar 3, 2026

fixed the git history and rebased onto develop; should be ready to review from my side.

Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Cool driver!

@hshose
Copy link
Contributor Author

hshose commented Mar 5, 2026

@salkinium thank you for your review ❤️

@hshose hshose force-pushed the feature/iwr6843aop branch from 7767efc to 18703a0 Compare March 5, 2026 18:23
@joelsa
Copy link
Contributor

joelsa commented Mar 5, 2026

Works perfectly on my radar!

image

@salkinium salkinium added this to the 2026q1 milestone Mar 6, 2026
@salkinium salkinium force-pushed the feature/iwr6843aop branch from 18703a0 to 39d34dc Compare March 7, 2026 17:46
Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

I squashed the last fixup commit into the correct commits. Thanks!

@salkinium salkinium enabled auto-merge (rebase) March 7, 2026 17:47
@salkinium salkinium merged commit 20a9705 into modm-io:develop Mar 7, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants