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: Device refactor to integrate Platform-Devices #814

Merged
merged 5 commits into from
Aug 30, 2024
Merged

Conversation

IgnoreWarnings
Copy link
Collaborator

@IgnoreWarnings IgnoreWarnings commented Aug 14, 2024

Rename (pci::)Device to PciDevice and move it to devices folder. Change DeviceList to PciDeviceList. Change namespace from pci to devices.
#808 Introduces a Device interface which will integrate Pci Devices and Platform Devices.

@IgnoreWarnings IgnoreWarnings changed the title Refactor: Rename Device to PciDevice; Move PciDevice to devices folder Refactor: Device to integrate with Platform-Devices Aug 23, 2024
@IgnoreWarnings IgnoreWarnings changed the title Refactor: Device to integrate with Platform-Devices Refactor: Device refactor to integrate Platform-Devices Aug 23, 2024
@IgnoreWarnings IgnoreWarnings force-pushed the refactor-pci branch 2 times, most recently from 1eb8d83 to d4015e8 Compare August 26, 2024 10:35
@n-eiling
Copy link
Member

@stv0g we are waiting for your approval :)

@stv0g
Copy link
Contributor

stv0g commented Aug 30, 2024

Glaube da sind irgendwie Zertifikate in der CI abgelaufen:

tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2024-08-29T15:02:11Z is after 2024-08-29T08:34:57Z

Copy link
Contributor

@stv0g stv0g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me :) Could you rename pci_device.hpp to just pci.hpp, fix the CI and remove to one superfluous parantheses?

Thanks :) 👍🏻

common/lib/CMakeLists.txt Show resolved Hide resolved
fpga/lib/pcie_card.cpp Show resolved Hide resolved
@n-eiling
Copy link
Member

I guess it was fixed. I fixed the CI by rerunning it.

@IgnoreWarnings
Copy link
Collaborator Author

Glaube da sind irgendwie Zertifikate in der CI abgelaufen:

tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2024-08-29T15:02:11Z is after 2024-08-29T08:34:57Z

@n-eiling Can you fix this?

@stv0g stv0g merged commit 83e95f8 into master Aug 30, 2024
2 checks passed
@stv0g stv0g deleted the refactor-pci branch August 30, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants