M-Labs
/
zynq-rs
10
0
Fork 12
Code Issues 6 Pull Requests 1 Releases Wiki Activity

GIC: fix wrong core target config when enabling interrupt #109

Merged
sb10q merged 1 commits from morgan/zynq-rs:irq_fix into master 2023-12-19 18:41:03 +08:00
Conversation 1 Commits 1 Files Changed 1
morgan commented 2023-12-19 17:20:17 +08:00
Owner

Summary

  • Before the patch, an extra 1 is added to target_cpu and the interrupt will be configured to the wrong CPU target.
target_cpu bits set before patch bits set after patch
core0 0b10 (enable interrupt on core1) 0b01 (enable interrupt on core0)
core1 0b11 (enable interrupt on core0 & core1) 0b10 (enable interrupt on core1)
# Summary - Before the patch, an extra 1 is added to `target_cpu` and the interrupt will be configured to the wrong CPU target. | target_cpu | bits set before patch | bits set after patch | | -----------| ----------- | ----------- | | core0 | 0b10 (enable interrupt on core1) | 0b01 (enable interrupt on core0) | | core1 | 0b11 (enable interrupt on core0 & core1)| 0b10 (enable interrupt on core1) | - [Correct ICDIPTR Register configuration from AMD](https://docs.xilinx.com/r/en-US/ug585-zynq-7000-SoC-TRM/Software-Generated-Interrupts-SGI?tocId=0TsxAmy8MHRPDsayG96K1Q)
👍 1
morgan added 1 commit 2023-12-19 17:20:18 +08:00
sb10q commented 2023-12-19 18:40:32 +08:00
Owner

PR title is good but the commit message should follow.

PR title is good but the commit message should follow.
sb10q merged commit 4168eb63a7 into master 2023-12-19 18:41:03 +08:00
morgan deleted branch irq_fix 2024-01-31 11:23:47 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/zynq-rs#109
There is no content yet.
Delete Branch "morgan/zynq-rs:irq_fix"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?