-
Notifications
You must be signed in to change notification settings - Fork 640
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
Upstreaming msm8916 SoC support #405
Comments
It would be great if useful part of CAF lk is merged to the mainline.
It is safer not to do such thing without permission by the original author. |
I wonder what constitutes "useful"? :) I have some LK out-of-tree code with basic working timers/interrupts/UART. I still hope to continue hacking on that some time in the future.
Yes, I suspect you're right. Relatedly, the downstream CAF LK code is ... well, it's a lot like the downstream Android kernel forks of old - stuff is just hacked in all over the place in inappropriate places. So, ultimately I think the "can we relicense or adopt this code?" question is somewhat moot because the code probably just needs to be mostly rewritten. That's basically what I've been doing in that linked repo, though I haven't been cautious about IP contamination. I assume that a permissive license like BSD means this shouldn't be too much of an issue, though? |
I meant platform-dependent, non-legacy part, probably. I didn't consider it tbh. |
Hi there @travisg . As I'm sure you're aware, the lk2nd project is an actively maintained LK derivative that was spun out from the CodeAurora (CAF) code. It's used as a chained (from the vendor LK) bootloader in the
boot
partition on a range of msm8916+msm8974+msm8226 devices.The CAF codebase was forked from LK a very long time ago. Consequently lk2nd looks "kind-of" like LK but as you noted it's diverged quite a lot.
As an educational exercise I recently started exploring porting that downstream work on top of modern LK upstream.
I've managed to get some basic clock + UART working on an msm8196 device in this commit: d213437
To be clear, I'm not asking you to review that code at all yet, since it's 5000 lines of mostly copy/pasta 😅
What I'm asking is if you'd be interested in seeing support for these qcom devices brought upstream? If so, I'd be happy to work with you on this, and massage the downstream code for style/quality/nits/etc to make that happen.
The most obvious issue I see is that the downstream code that came from the CAF repo is BSD-licensed, whereas LK upstream is purely MIT. I have no idea how to navigate such a thing. I've always regarded BSD as a very permissive license though. Interestingly, it seems that it might be possible to simply re-license the BSD code as MIT, as long as the original copyright notices are retained? I dunno. I am a computer person not a lawyer ;)
The text was updated successfully, but these errors were encountered: