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

Refactor: Split core make into multiple functions #791

Merged
merged 8 commits into from
Jun 27, 2024

Conversation

IgnoreWarnings
Copy link
Collaborator

Seperate the core factory make method into multiple parts to improve readability, reusability and mainainability.
Verified to work as expected on miob hardware.
In addition, #783 will implement features which require the seperation of parsing and initialization.

@IgnoreWarnings
Copy link
Collaborator Author

The e41651e commit isnt supposed to be in this PR, i will remove it tomorrow

common/include/villas/kernel/vfio_device.hpp Outdated Show resolved Hide resolved
fpga/lib/core.cpp Show resolved Hide resolved
fpga/lib/core.cpp Outdated Show resolved Hide resolved
fpga/lib/core.cpp Outdated Show resolved Hide resolved
@n-eiling
Copy link
Member

please fix DCOs and I would also like a review from @stv0g. I'm not sure if the logger use is correct.

@n-eiling n-eiling merged commit 54796d1 into VILLASframework:master Jun 27, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants