-
Notifications
You must be signed in to change notification settings - Fork 18
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
Abstract away board and toolchain details #216
base: main
Are you sure you want to change the base?
Conversation
Put the common bits in a makefile snippet. Signed-off-by: Peter Chubb <[email protected]>
Is this a good idea? |
I don't think it's worse like this but I'm not sure if it's an improvement either. Having the toolchain extracted makes more sense than the boards I think. There is a bit of duplication with the boards in each example but it's really simple, just for the device classes. |
I think that it is a good improvement imo. Especially when we change names of devices to their proper names (like meson eth changing to dwmac4.20), or have devices with peripherals of completely different names (like the star64 with snps-uart, dwmac5.1 eth), just having a central config for that specific board will be easier for users. But that's more from the perspective of LionsOS, where the tooling will eventually deal with all of this. After discussing with Ivan, maybe the board abstractions don't necessarily belong in the sDDF. |
I was mostly thinking about this for LionsOS, but would like to have the same mechanism everywhere, and this seemed the right place to start. |
I guess I don't really know what problem we're trying to solve? Why is the current approach inadequate? |
What I mean by this is that the simpler approach seems fine for sDDF example systems and the abstraction is probably better left solely to LionsOS. |
I'm not sure the current approach is scalable to many many boards. Merely including a board description file, rather than repeating a huge if-then-else chain means:
|
Put the common bits in a makefile snippet.