-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
bananaPi-p2Maker: Add support for Banana Pi P2 Maker #163
base: development
Are you sure you want to change the base?
Conversation
Hi! 👋 Given this is an Allwinner board, the changes are 99.99% likely good without requiring any more testing. In my experience they all work so much alike that it's (thankfully) boring. I don't see you specifically stating that you've tested HDMI out. Have you? Otherwise it all looks good. I don't really see any reason not to merge this.
I don't think I'll have the time to test, but I wonder if any h3/h5 boards I own have been affected. I guess the way to check is that USB plainly doesn't work? Or is it a particular interface? I'm led to understand that the OTG port at the very least would be affected.
It really should, I've not been stellar at it, but ideally only opinionated changes should live in this repo. Or at least I'd like that every changes sees a genuine effort at being sent upstream.
Assuming it's about the individuals, have you had experience with U-Boot specifically? You can leave the answer vague if you're not comfortable, and I definitely don't want names in that unlikely event. I've found that the worst that happened to me was that a set of patches was left ignored, though asking around on their IRC channel gives an almost direct link to some maintainers that can help. My only complaint is that it's the ol' patch-in-mail, everything else has been good. |
Yes. There's some weird artifacting at 4K without accelerated graphics, but it's still usable for setup.
I think it's just the OTG port, and only when device mode support is compiled out in U-Boot, but this device only has the one port and I didn't do much testing with device mode enabled.
It's not really about any individuals. |
9ac1482
to
e9d6d6a
Compare
Oops... Got in the flow with other PRs and rebased+pushed, even though this applies as-it-is on Can you verify this still works fine on the device with 2022.07? I've built it and it built without issues, without any change. Given how boring this update cycle (2021.10 → 2022.07) was for Allwinner, it wouldn't surprise me things just work. |
The Banana Pi P2 Maker is a low-cost Allwinner H2+ (variant of the H3) development board primarily notable for having (optional) PoE. I've tested this with Arch Linux ARM, Debian, and Fedora, and all three work. It likely also would work with the P2 Zero, which uses the same PCB but with eMMC (and possibly support components for it?) populated, but of course without eMMC support. I initially attempted implementing the P2 Zero instead of the P2 Maker, but I wouldn't have been able to test the eMMC capabilities, and the missing eMMC caused warnings during boot.
The U-Boot patch could probably be upstreamed (especially the change to
sunxi-h3-h5.dtsi
, which is necessary for USB host functionality to work in U-Boot after a refactor), but I've had bad experiences submitting patches to these projects.