-
Notifications
You must be signed in to change notification settings - Fork 228
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding NXP NavQPlus to detected boards #283
Conversation
…the chip is already detected) Future boards using the same SOC may need to edit the board name or detection line in bin/detect.py so it doesn't list it as an NXP NAVq board Original NavQ board not available to I couldn't add it too. Added for the Hovergames 3 Team Mcfly 2023 Changes to be committed: modified: adafruit_platformdetect/board.py modified: adafruit_platformdetect/constants/boards.py modified: bin/detect.py
@makermelissa Can you review this when you have time please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of this looks good. Just a quick question about an extra space.
…ions when I add it into Blinka Changes to be committed: modified: adafruit_platformdetect/board.py modified: adafruit_platformdetect/chip.py modified: adafruit_platformdetect/constants/chips.py
Can you run pre-commit so that it fixes the formatting and the checks pass? Otherwise, the code itself looks good. |
Sorry about that @makermelissa. I started looking at the Blinka code and found I need to detect this board separately. Otherwise some of the GPIO and UART numbers won't line up and I didn't want to risk breaking existing code. I didn't realize Github auto-adds to the Pull Request when I pushed in the new commits.
|
Awesome. Did you want me to wait before merging this? |
No, if you don't see any issues it can be merged. |
Thank you! |
dding the NXP NavQPlus board:
Board documentation available here:
(Based on the the NXP i.MX 8M Plus SOC so the chip is already detected)
Future boards using the same SOC may need to edit the board name or detection line in bin/detect.py so it doesn't list it as an NXP NavQ based board
Tested on Ubuntu 22.04 built on top of NXP Yocto 5.15
Added for the Hovergames 3
Team McFly
2023
Instructions followed : https://learn.adafruit.com/adding-a-single-board-computer-to-platformdetect-for-blinka/overview