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

Additional Constructor for ACCL XRT Driver #46

Open
Mellich opened this issue Apr 19, 2022 · 0 comments
Open

Additional Constructor for ACCL XRT Driver #46

Mellich opened this issue Apr 19, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@Mellich
Copy link
Contributor

Mellich commented Apr 19, 2022

In the XRT driver, the ACCL constructor currently requires the user to give the cclo and control kernel as xrt::ip and xrt::kernel objects.
But in some cases it may be easier for the user to assume default names for these kernels to allow reduction of the code required for initialization.

Therefore, an additional ACCL constructor taking the xrt::uuid instead of the kernels could be provided. The instantiation of the kernels is moved to the constructor itself. Optionally, the kernel names may be given as std::strings to the constructor.

@Mellich Mellich added the enhancement New feature or request label Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant