Skip to content

zephyr: sw_isr_table: Fix compilation error #92336

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions include/zephyr/sw_isr_table.h
Original file line number Diff line number Diff line change
Expand Up @@ -243,10 +243,12 @@ struct z_shared_isr_table_entry z_shared_sw_isr_table[];
__attribute__((section(sect))) \
__used _MK_IRQ_ELEMENT_NAME(func, __COUNTER__) = ((uintptr_t)(func)); \
), ( \
static void __attribute__((section(sect))) __attribute__((naked)) \
__used _MK_IRQ_ELEMENT_NAME(func, __COUNTER__)(void) { \
void __attribute__((section(sect))) __attribute__((naked)) \
/* clang-format off */ \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had to turn off clang-format here because it wanted to make line 248 and 249 a single line, which breaks the other check that lines are shorter than 100 characters.

Copy link
Collaborator

Choose a reason for hiding this comment

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

good thing, that clang-format isn't mandatory

__used _MK_IRQ_ELEMENT_NAME(func, __COUNTER__) (void) { \
__asm(ARCH_IRQ_VECTOR_JUMP_CODE(func)); \
} \
/* clang-format on */ \
))

#define Z_ISR_DECLARE_DIRECT_C(irq, flags, func, counter) \
Expand Down