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

Defining Mailbox Macro for QDMA to fix APU crash on RAVE #8052

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

bisingha-xilinx
Copy link
Collaborator

@bisingha-xilinx bisingha-xilinx commented Apr 4, 2024

QDMA blindly registers intr for mailbox even if its disabled on the hw.
NOTE: this is a workaround to move forward with this https://jira.xilinx.com/browse/CR-1193112
Once QDMA fixes the issue this commit should be reverted

Problem solved by the commit

https://jira.xilinx.com/browse/CR-1193112

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

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

It avoids QDMA to not register intr for mailbox

Risks (if any) associated the changes in the commit

none

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

Tested on RAVE and V70 with "xbutil validate -d"

Documentation impact (if any)

none

    QDMA blindly registers intr for mailbox even if its disabled on the hw.
    NOTE: this is a workaround to move forward with this https://jira.xilinx.com/browse/CR-1193112
    Once QDMA fixes the issue this commit should be reverted

Signed-off-by: bisingha <[email protected]>
@gbuildx
Copy link
Collaborator

gbuildx commented Apr 4, 2024

Can one of the admins verify this patch?

@chvamshi-xilinx
Copy link
Collaborator

add to whitelist

@chvamshi-xilinx chvamshi-xilinx merged commit 56c7e67 into master Apr 4, 2024
21 of 33 checks passed
@bisingha-xilinx bisingha-xilinx deleted the MailboxMacroForQDMA branch April 4, 2024 18:10
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.

3 participants