-
Notifications
You must be signed in to change notification settings - Fork 7.6k
drivers: intc_dw: support multiple instances #92268
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
base: main
Are you sure you want to change the base?
Conversation
Hello @afongTT, and thank you very much for your first pull request to the Zephyr project! |
INTC_INST_ISR_TBL_OFFSET(0), DT_INST_INTC_GET_AGGREGATOR_LEVEL(0)); | ||
#define INTC_DW_DEVICE_INIT(inst) \ | ||
\ | ||
static void dw_ictl_config_irq_##inst(const struct device *port) \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: the port
arg is probably not required in the future as well since we customized it for every instance
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the unused port
argument.
Update the initializers to support platforms with multiple instances of the DesignWare interrupt aggregator. Also ensure that the initialize function calls irq_enable(). The calculation of the _sw_isr_table entry also now takes CONFIG_GEN_IRQ_START_VECTOR into account. Signed-off-by: Aaron Fong <afong@tenstorrent.com>
|
Update the initializers to support platforms with multiple instances of the DesignWare interrupt aggregator. Also ensure that the initialize function calls
irq_enable()
. The calculation of the_sw_isr_table
entry also now takesCONFIG_GEN_IRQ_START_VECTOR
into account.