Skip to content

Add overlaycheck2 - #206

Merged
pelwell merged 2 commits into
raspberrypi:masterfrom
pelwell:overlaycheck2
Jul 29, 2026
Merged

Add overlaycheck2#206
pelwell merged 2 commits into
raspberrypi:masterfrom
pelwell:overlaycheck2

Conversation

@pelwell

@pelwell pelwell commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Reinvent overlaycheck as a Python utility that, for the most part,
replaces the pairing of ovmerge as a preprocessor and separate passes
for each test with the internalisation of the DT as a data structure
on which the various checkers work. The end result has been a new bug
found (apds9660 wasn't referencing its pinctrl node) and a two thirds
reduction in run time.

ovmerge.py is a reimplementation of ovmerge in Python. It should be
identical, output-wise, compared to the Perl original.

To enable the creation of related tools, the parsing and DT manipulation
parts of the utility have been placed in ovmerge_engine.py.

Signed-off-by: Phil Elwell <phil@raspberrypi.com>
@popcornmix

Copy link
Copy Markdown
Contributor

Moving form perl to python is a big plus for me.

Does overlaycheck2 miss any functionality over overlaycheck?
Is there a future plan to replace overlaycheck with overlaycheck2?

@pelwell

pelwell commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

overlaycheck2 is a full, drop-in replacement for overlaycheck. It returns identical results, except for the additional error it found. And yes, the plan is to replace overlaycheck with the new version.

Reinvent overlaycheck as a Python utility that, for the most part,
replaces the pairing of ovmerge as a preprocessor and separate passes
for each test with the internalisation of the DT as a data structure
on which the various checkers work. The end result has been a new bug
found (apds9660 wasn't referencing its pinctrl node) and a two thirds
reduction in run time.

Signed-off-by: Phil Elwell <phil@raspberrypi.com>
@pelwell
pelwell merged commit 12ac746 into raspberrypi:master Jul 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants