-
Notifications
You must be signed in to change notification settings - Fork 972
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
Update Raspberry Pi 3 device tree list #3301
Conversation
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.
This needs rebase after RPi kernel bump to 6.6 😬
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Make sure all Raspberry Pi devices using the BCM2837 SoC (or the SiP version RP3A0 of it) are deployed. This especially makes sure that we only deploy the downstream device trees (named bcm2710*) and deploy device trees for the CM3 as well as the Zero 2 and Zero 2 W consistently for 32-bit and 64-bit.
3ff9fe2
to
d0ebfb4
Compare
Right! Rebased! |
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.
Looks good. Seems that the broadcom/bcm2837-rpi-3-b
wasn't needed, it's the upstream DT with some downstream parts missing. Unless it was picked by FW in some cases (which IMHO doesn't make much sense) it should be safe to remove it.
I've just checked on a Raspberry Pi 3 (non-plus), and it doesn't seem to get picked today ( |
Hi @agners, thank you soo much for the fast resolution! |
device trees are correcty deployed. (cf. home-assistant/operating-system#3301)
Make sure all Raspberry Pi devices using the BCM2837 SoC (or the SiP version RP3A0 of it) are deployed. This especially makes sure that we only deploy the downstream device trees (named bcm2710*) and deploy device trees for the CM3 as well as the Zero 2 and Zero 2 W consistently for 32-bit and 64-bit.
Fixes: #3298