Skip to content

pkg/u8g2: implement disp_dev#21725

Merged
crasbe merged 2 commits intoRIOT-OS:masterfrom
leandrolanzieri:pr/u8g2_disp_dev
Sep 18, 2025
Merged

pkg/u8g2: implement disp_dev#21725
crasbe merged 2 commits intoRIOT-OS:masterfrom
leandrolanzieri:pr/u8g2_disp_dev

Conversation

@leandrolanzieri
Copy link
Copy Markdown
Contributor

@leandrolanzieri leandrolanzieri commented Sep 17, 2025

Contribution description

This implements our Display Device Generic API for the u8g2 package. The main reason for this is to enable using these displays with high-level libraries, such as LVGL (see #21726).

Testing procedure

You can try the tests/drivers/disp_dev test application. So far I've tested the implementation with OLED monochrome displays based on the 1306 driver, both SPI and I2C versions are working.

Issues/PRs references

Related to #21726

@leandrolanzieri leandrolanzieri added the Type: new feature The issue requests / The PR implemements a new feature for RIOT label Sep 17, 2025
@github-actions github-actions bot added Area: doc Area: Documentation Area: pkg Area: External package ports Area: drivers Area: Device drivers Area: sys Area: System labels Sep 17, 2025
@leandrolanzieri leandrolanzieri changed the title pkg/u8g2: disp dev pkg/u8g2: implement disp_dev Sep 17, 2025
@crasbe crasbe added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 17, 2025
@riot-ci
Copy link
Copy Markdown

riot-ci commented Sep 17, 2025

Murdock results

✔️ PASSED

ec9548d pkg/u8g2: add disp_dev support

Success Failures Total Runtime
10516 0 10516 14m:04s

Artifacts

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 did not go through the actual code, this is just styling related.

@leandrolanzieri
Copy link
Copy Markdown
Contributor Author

Thanks for checking @crasbe, I think I addressed everything.

@leandrolanzieri
Copy link
Copy Markdown
Contributor Author

I restructured the docs

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.

LGTM, please squash :)

@crasbe crasbe enabled auto-merge September 18, 2025 12:09
@crasbe crasbe added this pull request to the merge queue Sep 18, 2025
Merged via the queue into RIOT-OS:master with commit f72ddd7 Sep 18, 2025
25 checks passed
@leandrolanzieri
Copy link
Copy Markdown
Contributor Author

Thanks for the review!

@leandrolanzieri leandrolanzieri deleted the pr/u8g2_disp_dev branch September 19, 2025 07:54
@benpicco benpicco added this to the Release 2025.10 milestone Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: doc Area: Documentation Area: drivers Area: Device drivers Area: pkg Area: External package ports Area: sys Area: System 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

Development

Successfully merging this pull request may close these issues.

4 participants