Skip to content
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

add dtb support for RPI CM4, CM4s, CM4_ioBoard #1182

Closed
wants to merge 1 commit into from

Conversation

klaus4
Copy link
Contributor

@klaus4 klaus4 commented Apr 18, 2024

Since the test of PR #1179 was successful
it should make sense to support these models.
That wasn't possible before because cm4-models crashed by pcie devmatch .

While in my tests I had manually added only bcm2711-rpi-cm4.dtb to config.txt
without the corresponding bcm2711-rpi-cm4-io.dtb for the io-board,
I presume there normally should be no caveats with bcm2711-rpi-cm4-io.dtb .

@emaste
Copy link
Member

emaste commented Apr 18, 2024

This looks reasonable, thank you for this. Can you please include a Signed-off-by: trailer in the commit, certifying agreement with https://developercertificate.org/.

Signed-off-by: Klaus Küchemann <maciphone2@googlemail.com>
@klaus4 klaus4 reopened this Apr 19, 2024
@klaus4
Copy link
Contributor Author

klaus4 commented Apr 19, 2024

..... Can you please include a Signed-off-by: trailer in the commit....

thank you for the tip and sorry for close/reopen- accidentally

@bsdimp bsdimp self-assigned this Apr 19, 2024
@bsdimp
Copy link
Member

bsdimp commented Apr 19, 2024

merged. thanks for the contribution... the other rpi one is waiting on ed making contact with the broadcom linux kernel engineer, if possible...

@bsdimp bsdimp closed this Apr 19, 2024
freebsd-git pushed a commit that referenced this pull request Apr 19, 2024
Signed-off-by: Klaus Küchemann <maciphone2@googlemail.com>
Reviewed by: imp
Pull Request: #1182
@klaus4
Copy link
Contributor Author

klaus4 commented Apr 19, 2024

thank you !
should be noted that without #1179 the CM4-models will panic by default,
with or without physical pcie-device connected.
can be handled for CM4 users by a temporarily devmatch_enable=„NO“ in case that #1179 won`t reach the next snapshot.
Regards

@bsdimp
Copy link
Member

bsdimp commented Apr 19, 2024

Should i revert until Ed can land #1179?

@klaus4
Copy link
Contributor Author

klaus4 commented Apr 20, 2024

perhaps better reverting (only if Ed thinks it could last a bit longer to land #1179) .

@klaus4
Copy link
Contributor Author

klaus4 commented Apr 20, 2024

but perhaps prefer to land #1179 now ,…
while of course it`s always good contacting a broadcom engineer..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants