onl: allow building more platforms #147
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
ideaship
requested changes
Jan 31, 2024
But if it makes you happy I can add the link to the docs in the commit. |
|
I was hoping for something like: if you want to use this cool new feature, here's how you could do it. I like having such information in the commit message, even though there are other places for documentation. But maybe you are right. Everybody who can build BISDN Linux can figure this out. |
Instead silencing each instance of a ignored result, disable treating it as an error and drop all patches modifying code. All we do is just ignoring the result anyway, so we don't lose anything by disabling the error. Keep the patches that actually do some error checking in these cases though. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
Replace individual module update patches with patches updating all drivers in the tree. No functional changes apart from context changes due to reordering of patches. Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
Add PACKAGECOFIG[1] support for optionally building more platforms: - "unsupported" - buildable, but untested - "broken" - does not build (mostly as "known broken" list) Due to the way getVar works, we need to define the appropriate lists even when they are empty. By default stay at the supported/tested level. Broken for now also includes platforms that technically build, but aren't handled by our recipe correctly, as it assumes all platforms have modules and a platform library, which isn't true for some platforms. [1] https://docs.yoctoproject.org/ref-manual/variables.html#term-PACKAGECONFIG Signed-off-by: Jonas Gorski <jonas.gorski@bisdn.de>
KanjiMonster
force-pushed
the
jogo_onl_more_platforms
branch
from
965eb6b
to
15779b6
Compare
ideaship
approved these changes
Feb 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To make it easier adding new platform support make it optional to build untested platforms (which are not included in the image).
This way these can be then installed on the device via opkg and and any further changes can be done on the device.
By default, do not build these platforms, but we may chose to do that for nightly builds.
Since building more platforms means needing more fixes, consolidate the linux 6.1 API update patches into one, and pass appropriate build options to ignore some warnings instead of silencing in the code, allowing us to reduce the amount of patches needed further.