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

add support for mem_banks for --add-pskernel #7606

Merged
merged 3 commits into from
Jul 14, 2023
Merged

Conversation

xfreid
Copy link
Collaborator

@xfreid xfreid commented Jun 28, 2023

Problem solved by the commit

Add support for mem_banks for --add-pskernel

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

N/A

How problem was solved, alternative solutions (if any) and why they were rejected

N/A

Risks (if any) associated the changes in the commit

None

What has been tested and how, request additional testing if necessary

Manual and unit test

Documentation impact (if any)

None

@xfreid xfreid requested a review from rozumx as a code owner June 28, 2023 15:45
@gbuildx
Copy link
Collaborator

gbuildx commented Jun 28, 2023

Can one of the admins verify this patch?

@xfreid
Copy link
Collaborator Author

xfreid commented Jun 28, 2023

Currently, the output looks like the following

Specifying memory banks in --add-pskernel is an adanced feature
connecting arg 0 to mem bank 0
connecting arg 0 to mem bank 1
connecting arg 1 to mem bank 0
connecting arg 1 to mem bank 1
connecting arg 2 to mem bank 0
connecting arg 2 to mem bank 1
connecting arg 6 to mem bank 0
connecting arg 6 to mem bank 1

@xfreid
Copy link
Collaborator Author

xfreid commented Jun 29, 2023

retest this please

1 similar comment
@dayeh-xilinx
Copy link

retest this please

@gbuildx
Copy link
Collaborator

gbuildx commented Jun 29, 2023

Build Passed!

Copy link
Collaborator

@rozumx rozumx left a comment

Choose a reason for hiding this comment

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

Some minor changes are being requested.

@xfreid xfreid requested a review from rozumx July 13, 2023 19:37
@xfreid
Copy link
Collaborator Author

xfreid commented Jul 13, 2023

retest this please

@manikandan-xilinx
Copy link
Collaborator

retest this please.

@gbuildx
Copy link
Collaborator

gbuildx commented Jul 13, 2023

Build failed :(

@xfreid
Copy link
Collaborator Author

xfreid commented Jul 13, 2023

retest this please

Copy link
Collaborator

@rozumx rozumx left a comment

Choose a reason for hiding this comment

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

Thank you for making these changes.

@rozumx
Copy link
Collaborator

rozumx commented Jul 13, 2023

retest this please

@manikandan-xilinx
Copy link
Collaborator

retest this please.

@gbuildx
Copy link
Collaborator

gbuildx commented Jul 14, 2023

Build Passed!

@rozumx rozumx merged commit c895c65 into Xilinx:master Jul 14, 2023
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.

6 participants