Skip to content

drivers/paa5100je: support for the PAA5100JE and PMW3901 optical flow sensors#21856

Merged
crasbe merged 1 commit intoRIOT-OS:masterfrom
LeonardHerbst:paa5100je_driver
Nov 7, 2025
Merged

drivers/paa5100je: support for the PAA5100JE and PMW3901 optical flow sensors#21856
crasbe merged 1 commit intoRIOT-OS:masterfrom
LeonardHerbst:paa5100je_driver

Conversation

@LeonardHerbst
Copy link
Copy Markdown
Contributor

@LeonardHerbst LeonardHerbst commented Nov 6, 2025

Contribution description

This driver adds support for the PMW3901 and PAA5100JE flow sensors. The PAA5100JE is used for near optical flow while the PMW3901 is used with drones, etc.
Both variations of the sensor can be used at the same time.
This driver was adapted from this python driver.

Testing procedure

I ran the test I created using a PA5100JE breakout board and the bitcraze flow deck v2.

@github-actions github-actions bot added Area: doc Area: Documentation Area: tests Area: tests and testing framework Area: drivers Area: Device drivers Area: SAUL Area: Sensor/Actuator Uber Layer Area: Kconfig Area: Kconfig integration labels Nov 6, 2025
@crasbe crasbe added Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 6, 2025
@crasbe
Copy link
Copy Markdown
Contributor

crasbe commented Nov 6, 2025

Can you squash the commits before I start a review?

@LeonardHerbst
Copy link
Copy Markdown
Contributor Author

LeonardHerbst commented Nov 6, 2025

Can you squash the commits before I start a review?

Thanks for the quick response! The commits are squashed now.

crasbe
crasbe reviewed Nov 6, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@crasbe crasbe left a comment

Choose a reason for hiding this comment

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

You will also need a Makefile.ci:

BOARD_INSUFFICIENT_MEMORY := \
    atmega8 \
    #

@riot-ci
Copy link
Copy Markdown

riot-ci commented Nov 6, 2025 •
edited
Loading

Murdock results

✔️ PASSED

fb4e183 drivers: added support for the PA5100JE and PMW3901 optical flow sensor

Success Failures Total Runtime
10932 0 10933 09m:53s

Artifacts

crasbe
crasbe approved these changes Nov 7, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@crasbe crasbe left a comment

Choose a reason for hiding this comment

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

I can't test it because I don't have the hardware, but it looks good.

The naming is a bit suboptimal because it covers two different sensors, but I couldn't come up with anything better.

Please squash :)

@LeonardHerbst
Copy link
Copy Markdown
Contributor Author

LeonardHerbst commented Nov 7, 2025 •
edited
Loading

I can't test it because I don't have the hardware, but it looks good.

The naming is a bit suboptimal because it covers two different sensors, but I couldn't come up with anything better.

Please squash :)

Yeah that also bugs me but pixartflow or anything like this is way to broad. Thanks for reviewing :)

@crasbe crasbe enabled auto-merge November 7, 2025 13:32
@crasbe crasbe added this pull request to the merge queue Nov 7, 2025
Merged via the queue into RIOT-OS:master with commit 64d3cea Nov 7, 2025
25 checks passed
@LeonardHerbst LeonardHerbst deleted the paa5100je_driver branch December 1, 2025 09:46
@leandrolanzieri leandrolanzieri added this to the Release 2026.01 milestone Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crasbe crasbe crasbe approved these changes

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss MrKevinWeiss is a code owner

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

Assignees

No one assigned

Labels

Area: doc Area: Documentation Area: drivers Area: Device drivers Area: Kconfig Area: Kconfig integration Area: SAUL Area: Sensor/Actuator Uber Layer Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Milestone

Release 2026.01

Development

Successfully merging this pull request may close these issues.

4 participants

@LeonardHerbst @crasbe @riot-ci @leandrolanzieri