Skip to content

Fix PocketBeagle 2 support: correct DT compatible string and black formatting#411

Merged
makermelissa merged 2 commits intoadafruit:mainfrom
makermelissa-piclaw:fix-pocketbeagle2-pr409
Apr 22, 2026
Merged

Fix PocketBeagle 2 support: correct DT compatible string and black formatting#411
makermelissa merged 2 commits intoadafruit:mainfrom
makermelissa-piclaw:fix-pocketbeagle2-pr409

Conversation

@makermelissa-piclaw
Copy link
Copy Markdown
Contributor

@makermelissa-piclaw makermelissa-piclaw commented Apr 22, 2026

Fixes CI failure in #409 to get the PocketBeagle 2 support merged.

Changes

1. Fix black formatting in boards.py

Removed a stray blank line inside the BEAGLEBONE tuple that caused the black pre-commit check to fail (the CI failure).

2. Clean up comment in board.py

The special-case EEPROM check for PocketBeagle 2 is the correct approach — new-format Beagle EEPROMs (like BeaglePlay) have binary prefix bytes that can't be decoded as meaningful ASCII for the dictionary lookup. Replaced the uncertain comment with an explanatory one.

Builds on top of #409 by @explodeo.

explodeo and others added 2 commits February 21, 2026 22:46
- Fix boards.py: Remove stray blank line inside BEAGLEBONE tuple that
  caused black formatting CI failure.
- Clean up board.py comment: The special-case EEPROM check is the
  correct approach for new-format Beagle boards (like BeaglePlay),
  so replaced the uncertain comment with an explanatory one.
Copy link
Copy Markdown
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@makermelissa makermelissa merged commit 3e042bf into adafruit:main Apr 22, 2026
1 check 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